-
Notifications
You must be signed in to change notification settings - Fork 13.5k
Make -Ztrack-diagnostics emit like a note #143286
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: master
Are you sure you want to change the base?
Conversation
rustbot has assigned @WaffleLapkin. Use |
It's now emitted as a note into the json output as well, right? Could you then migrate tests using
|
b528aba
to
dc4877b
Compare
No, I only made it so that it gets rendered like a
Since this isn't changing the json, would you still like me to do this? |
This comment has been minimized.
This comment has been minimized.
Compiletest is working on the json output, so |
dc4877b
to
e47cd86
Compare
Some changes occurred in src/tools/clippy cc @rust-lang/clippy |
e47cd86
to
6d3ffe1
Compare
I don't know how I would add it to the JSON output, but if I need to, I'll see what I can figure out. |
#t-compiler/diagnostics > Rendering -Ztrack-diagnostics like a note
As discussed on the Zulip thread above, I want to make
-Ztrack-diagnostics
emit like anote
. This is because I find its current output jarring, and the fact that it gets rendered completely left-aligned, even in the middle of a snippet, seems like something that should be changed. Turning it into anote
seems like the best choice, as it would align it with the rest of the output, andnote
is already used for somewhat similar things, like seeing why a lint was fired.Note: turning
-Ztrack-diagnostics
into anote
will also makeannotate-snippets
API a bit cleaner