Skip to content

Enqueue additional and analyzer config documents in WorkCoordinator#58479

Merged
mavasani merged 2 commits intodotnet:mainfrom
mavasani:WorkcoordinatorFix
Dec 24, 2021
Merged

Enqueue additional and analyzer config documents in WorkCoordinator#58479
mavasani merged 2 commits intodotnet:mainfrom
mavasani:WorkcoordinatorFix

Conversation

@mavasani
Copy link
Copy Markdown
Contributor

Fixes #53192
Currently, work coordinator only enqueues source document IDs for project analysis work items. This means we don't analyze the additional documents, which can now report standalone diagnostics through RegisterAdditionalFileAction. This PR fixes this by enqueuing project's additional document IDs and analyzer config document IDs.

Fixes dotnet#53192
Currently, work coordinator only enqueues source document IDs for project analysis work items. This means we don't analyze the additional documents, which can now report standalone diagnostics through RegisterAdditionalFileAction. This PR fixes this by enqueuing project's additional document IDs and analyzer config document IDs.
@mavasani mavasani added this to the 17.1 milestone Dec 24, 2021
@mavasani mavasani requested review from a team and sharwell December 24, 2021 05:37
@mavasani mavasani merged commit 957f8b3 into dotnet:main Dec 24, 2021
@mavasani mavasani deleted the WorkcoordinatorFix branch December 24, 2021 16:20
@ghost ghost modified the milestones: 17.1, Next Dec 24, 2021
@Cosifne Cosifne modified the milestones: Next, 17.1.P3 Jan 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Roslyn Analyzer not clearing last diagnostic

3 participants