-
Notifications
You must be signed in to change notification settings - Fork 12.7k
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
E0220 needs to be updated to new format #35385
Labels
A-diagnostics
Area: Messages for errors, warnings, and lints
E-easy
Call for participation: Easy difficulty. Experience needed to fix: Not much. Good first issue.
E-mentor
Call for participation: This issue has a mentor. Use #t-compiler/help on Zulip for discussion.
Comments
sophiajt
added
A-diagnostics
Area: Messages for errors, warnings, and lints
E-easy
Call for participation: Easy difficulty. Experience needed to fix: Not much. Good first issue.
E-mentor
Call for participation: This issue has a mentor. Use #t-compiler/help on Zulip for discussion.
labels
Aug 5, 2016
sophiajt
pushed a commit
to sophiajt/rust
that referenced
this issue
Aug 7, 2016
Update E0220 message to new format Part of rust-lang#35233 . Fixes rust-lang#35385. r? @jonathandturner Should it keep E0191?
Requirements changed a little bit, instead of
a more semantic and still short phrase is:
NOTE: the full error label is still included on the top part aside to E0220 |
sophiajt
pushed a commit
to sophiajt/rust
that referenced
this issue
Aug 13, 2016
Update E0220 message to new format Part of rust-lang#35233 . Fixes rust-lang#35385. r? @jonathandturner Should it keep E0191?
eddyb
added a commit
to eddyb/rust
that referenced
this issue
Aug 14, 2016
Update E0220 message to new format Part of rust-lang#35233 . Fixes rust-lang#35385. r? @jonathandturner Should it keep E0191?
@chamoysvoice - looks like you need to update some of the unit tests so this can pass:
You can run the full compile-fail test suite to make sure the changes worked using:
|
bors
added a commit
that referenced
this issue
Oct 2, 2016
Update E0220 error format @jonathandturner Part of #35233 . Fixes #35385.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
A-diagnostics
Area: Messages for errors, warnings, and lints
E-easy
Call for participation: Easy difficulty. Experience needed to fix: Not much. Good first issue.
E-mentor
Call for participation: This issue has a mentor. Use #t-compiler/help on Zulip for discussion.
From: src/test/compile-fail/E0220.rs
Error E0220 needs a span_label, updating it from:
To:
The text was updated successfully, but these errors were encountered: