-
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
E0004 needs to be updated to new format #35191
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
E-easy
Call for participation: Easy difficulty. Experience needed to fix: Not much. Good first issue.
A-diagnostics
Area: Messages for errors, warnings, and lints
E-mentor
Call for participation: This issue has a mentor. Use #t-compiler/help on Zulip for discussion.
and removed
E-mentor
Call for participation: This issue has a mentor. Use #t-compiler/help on Zulip for discussion.
labels
Aug 2, 2016
I'll give this one a shot |
I wouldn't mind working on the bonus part of this issue too but i'm not sure what to change the span to. To me, it might make more sense to change it to
or
|
Yeah, I like:
|
Alright, i'll try implementing that |
sophiajt
pushed a commit
to sophiajt/rust
that referenced
this issue
Aug 5, 2016
Update E0004 to use labels Fixes rust-lang#35191 and is part of rust-lang#35233 "r? @jonathandturner
eddyb
added a commit
to eddyb/rust
that referenced
this issue
Aug 6, 2016
Update E0004 to use labels Fixes rust-lang#35191 and is part of rust-lang#35233 "r? @jonathandturner
I did not, and it looks like you already submitted a pull request for it? |
bors
added a commit
that referenced
this issue
Aug 8, 2016
Lengthen the span label to include match and expr for E0004 Part of #35233. Extension of #35191. r? @GuillaumeGomez
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/E0004.rs
E0004 needs a span_label, taking it from:
to:
Bonus: We may want to update the span that this error uses as well to point at something other than the beginning of the match.
The text was updated successfully, but these errors were encountered: