Skip to content
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

[TECH-228] Remnants of TECH-224 #12

Merged
merged 5 commits into from
Feb 28, 2023
Merged

[TECH-228] Remnants of TECH-224 #12

merged 5 commits into from
Feb 28, 2023

Conversation

asimmon
Copy link
Member

@asimmon asimmon commented Feb 28, 2023

  • Fixed: Rule GMDTR09 "Handlers should not call other handlers" was not applied when using our own new "Async"-suffixed extension methods, added more tests
  • Do what’s Microsoft is doing: Using ImmutableHashSet<T> in analyzers instead of HashSet<T>
  • Ignore more Renovate dependencies update. We don’t want to upgrade some packages (Microsoft.CodeAnalysis.* and BCL async interfaces) because we want to set a minimal required dependency

@asimmon asimmon requested a review from a team as a code owner February 28, 2023 19:28
@asimmon asimmon merged commit fc70b16 into main Feb 28, 2023
@asimmon asimmon deleted the feature/tech-228 branch February 28, 2023 21:35
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.

None yet

3 participants