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

Clear prior build errors for additional and analyzer config docs at s… #58457

Merged
merged 1 commit into from
Dec 22, 2021

Conversation

mavasani
Copy link
Contributor

…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.

…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.
@mavasani mavasani added this to the 17.1 milestone Dec 22, 2021
@mavasani mavasani requested review from NTaylorMullen, dibarbet and a team December 22, 2021 05:26
@mavasani
Copy link
Contributor Author

@dibarbet @NTaylorMullen do we have integration tests for razor? If so, I can add a regression test for this scenario.

@NTaylorMullen
Copy link
Contributor

@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

@mavasani
Copy link
Contributor Author

Alright, I’ll merge the PR and add a regression test later if feasible.

@mavasani mavasani merged commit 6ea8af0 into dotnet:main Dec 22, 2021
@ghost ghost modified the milestones: 17.1, Next Dec 22, 2021
@mavasani mavasani deleted the RazorBug branch December 22, 2021 16:15
@dibarbet
Copy link
Member

dibarbet commented Jan 3, 2022

@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

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?

@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.

4 participants