Clear prior build errors for additional and analyzer config docs at s…#58457
Clear prior build errors for additional and analyzer config docs at s…#58457mavasani merged 1 commit intodotnet:mainfrom
Conversation
…tart of build Fixes [AB#1424584](https://devdiv.visualstudio.com/DevDiv/_workitems/edit/1424584) The primary issue is that the lingering compiler warning CS0169 from prior solution snapshot is reported in `Index.razor`, which is an Additional document, not a regular source document. Existing code in clearing build diagnostics at start of a build was only clearing diagnostics with location in source documents. Doing the same for additional documents fixes this issue.
|
@dibarbet @NTaylorMullen do we have integration tests for razor? If so, I can add a regression test for this scenario. |
I'm actually not entirely sure. @dibarbet would know better though |
|
Alright, I’ll merge the PR and add a regression test later if feasible. |
Unfortunately not - the last time I attempted to go down that path the VM images did not have the required razor templates installed on them. @ryanbrandenburg I think you were recently working on razor integration tests - are the templates available now? |
…tart of build
Fixes AB#1424584
The primary issue is that the lingering compiler warning CS0169 from prior solution snapshot is reported in
Index.razor, which is an Additional document, not a regular source document. Existing code in clearing build diagnostics at start of a build was only clearing diagnostics with location in source documents. Doing the same for additional documents fixes this issue.