-
Notifications
You must be signed in to change notification settings - Fork 160
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fixed complaints for CA1825 #794
Conversation
src/Messaging/test/MessagingBase.Test/Handler/DestinationPatternsMessageConditionTest.cs
Show resolved
Hide resolved
@TimHess I've added Tried to reset and kept changes so it would all be in one commit, but it looks like it created 2 commits and 1 merge instead.. |
Sorry about the confusion there - I had merged another (important, pipeline-related) commit in main and then pushed the update button here.. I'll squash during the merge so it'll only be 1 commit in the end and nothing to worry about |
/azp run Steeltoe.All |
Azure Pipelines successfully started running 1 pipeline(s). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks!
SonarCloud Quality Gate failed. |
Modified all occurrences of zero-length array allocations to use Array.Empty().
Addresses #791