-
Notifications
You must be signed in to change notification settings - Fork 5.2k
Remove duplicate SourceGeneration generator refs #75083
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
Conversation
Unblocks #75058 The listed PR brings in a change in roslyn that errors for duplicate analyzer references. There were only two duplicates in the libraries source projects which are being removed. The actual reference is already brought in via generators.targets.
stephentoub
left a comment
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.
Thanks!
|
Tagging subscribers to this area: @dotnet/area-infrastructure-libraries Issue DetailsUnblocks #75058 The listed PR brings in a change in roslyn that errors for duplicate analyzer references. There were only two duplicates in the libraries source projects which are being removed. The actual reference is already brought in via generators.targets.
|
|
Failures are Linux arm64 process terminations, unrelated. |
|
Thanks @ViktorHofer. It looks like there may be another duplicate reference to Could we port this change to |
|
/backport to release/7.0 |
|
Started backporting to release/7.0: https://github.com/dotnet/runtime/actions/runs/3001098463 |
|
I've started a backport to |
Unblocks #75058
The listed PR brings in a change in roslyn that errors for duplicate analyzer references. There were only two duplicates in the libraries source projects which are being removed. The actual reference is already brought in via generators.targets.
Long term for inbox projects, we should agree on either explicitly referencing analyzers (what is currently being done for the JSImportGenerator) or automatically referencing them based on other inputs (currently, LibraryImportGenerator). Mixing both modes caused this issue but also makes it hard to reason about which analyzers run.