-
Notifications
You must be signed in to change notification settings - Fork 146
Initial draft of diagnostic redesign #3145
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
base: main
Are you sure you want to change the base?
Conversation
@IEvangelist This is ready. I may need to adjust the category names and any invalid wording I'm using due to lack of knowledge. |
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.
I like it a lot, well done! I do want @davidfowl, @eerhardt, and @mitchdenny to see this and provide feedback.
ASPIREAZURE001 | ||
``` | ||
|
||
## Suppress diagnostic |
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.
Where does all of this content move to? Maybe just create a new file with this under the Overview and put this suppression stuff in there?
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.
I copied it out and I'll update the .NET docs with the information.
I like splitting this out too. Can we confirm that the URLs given by the analyzers will still get people to the right place? |
Summary
Going to need help with wording.
Fixes #3143
Fixes #3142
Fixes #3141
Fixes #3140
Fixes #3139
Fixes #3131
Fixes #3126
Fixes #3057
Fixes #1658
TODO:
Internal previews