The Wayback Machine - https://web.archive.org/web/20220128212129/https://github.com/dotnet/aspnetcore/pull/39845
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

Do not upgrade warnings to errors in VS #39845

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Do not upgrade warnings to errors in VS #39845

wants to merge 1 commit into from

Conversation

@pranavkm
Copy link
Contributor

@pranavkm pranavkm commented Jan 28, 2022

Now that we have enabled a bunch of code analysis, we've discovered it adds friction to the dev process. This change aims to reduce the friction by preventing warnings to be upgraded to errors in VS. This way you can continue developing, while possible issues are still surfaced. E.g. from a recent error:

image

Now that we have enabled a bunch of code analysis, we've discovered it adds friction to the dev process. This change aims to reduce the friction by preventing warnings to be upgraded to errors in VS. This way you can continue developing, while possible issues are still surfaced. E.g. from a recent error:
@pranavkm pranavkm requested a review from as a code owner Jan 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants