Skip to content

Cleanup Csproj#869

Merged
mtmk merged 2 commits into
nats-io:mainfrom
thompson-tomo:CsprojCleanup
May 29, 2025
Merged

Cleanup Csproj#869
mtmk merged 2 commits into
nats-io:mainfrom
thompson-tomo:CsprojCleanup

Conversation

@thompson-tomo

Copy link
Copy Markdown
Contributor

Increase usage of directory.build.props to reduce replication of properties across files. Have also move to suppression in code rather than global,

@mtmk

mtmk commented May 24, 2025

Copy link
Copy Markdown
Member

quick peek looks good. thanks @thompson-tomo. will have a proper look soon just to double check

@thompson-tomo

Copy link
Copy Markdown
Contributor Author

No worries and the dotnet format issue should be resolved now.

@thompson-tomo

Copy link
Copy Markdown
Contributor Author

Flaky test again?

Signed-off-by: James Thompson <thompson.tomo@outlook.com>
Signed-off-by: James Thompson <thompson.tomo@outlook.com>

@mtmk mtmk left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM thanks @thompson-tomo cleaned up the proj files really nicely. sorry for the delay.

@mtmk
mtmk merged commit 9c2f2bb into nats-io:main May 29, 2025
@thompson-tomo
thompson-tomo deleted the CsprojCleanup branch May 30, 2025 00:17
mtmk added a commit that referenced this pull request Jun 30, 2025
* Fix inbox collection was modified exception (#885)
* Cleanup Csproj (#869)
* Add RequestReplyMode for JetStream (#867)
@mtmk mtmk mentioned this pull request Jun 30, 2025
mtmk added a commit that referenced this pull request Jun 30, 2025
* Fix inbox collection was modified exception (#885)
* Cleanup Csproj (#869)
* Add RequestReplyMode for JetStream (#867)
@mtmk mtmk mentioned this pull request Sep 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants