Skip to content

Conversation

@T-Gro
Copy link
Member

@T-Gro T-Gro commented Apr 18, 2023

The unification leads to attachment of cross-cutting functionality, especially telemetry collection.
At the same time, it reduces boilerplate and repetitive calls (see the lines added vs. lines removed)

@T-Gro T-Gro changed the title Codefixes cleanup - simplify dealing with diagnosticFilters Codefixes cleanup - cleanup fixableDiagnosticIds, unify more CodeFix registration calls Apr 18, 2023
@T-Gro
Copy link
Member Author

T-Gro commented Apr 19, 2023

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 2 pipeline(s).

@T-Gro T-Gro marked this pull request as ready for review April 19, 2023 08:42
@T-Gro T-Gro requested a review from a team as a code owner April 19, 2023 08:42
Copy link
Contributor

@psfinaki psfinaki left a comment

Choose a reason for hiding this comment

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

Awesome stuff!

@T-Gro T-Gro merged commit c2f7859 into dotnet:main Apr 19, 2023
@T-Gro T-Gro deleted the codefixes-cleanup branch April 19, 2023 11:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

5 participants