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.

Sign up for GitHub

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

Jump to bottom

--error-format=short should be an unstable option #45995

Closed
nrc opened this issue Nov 15, 2017 · 4 comments
Closed

--error-format=short should be an unstable option #45995

nrc opened this issue Nov 15, 2017 · 4 comments
Assignees
@GuillaumeGomez
Labels
A-diagnostics Area: Messages for errors, warnings, and lints P-high High priority T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.
Milestone
1.23

Comments

@nrc
Copy link
Member

nrc commented Nov 15, 2017

i.e., require -Zunstable-features. This should be backported to beta if necessary.

cc @GuillaumeGomez @alexcrichton @estebank
@nrc nrc added A-diagnostics Area: Messages for errors, warnings, and lints I-nominated T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. labels Nov 15, 2017
@nrc nrc mentioned this issue Nov 15, 2017
Merged
@Mark-Simulacrum Mark-Simulacrum added P-high High priority P-medium Medium priority and removed P-high High priority labels Nov 15, 2017
@Mark-Simulacrum
Copy link
Member

Setting priority to medium -- ideally we'd get this done before beta branched, but it definitely needs to land before 1.23 stable releases.

@Mark-Simulacrum Mark-Simulacrum added this to the 1.23 milestone Nov 15, 2017
@GuillaumeGomez GuillaumeGomez self-assigned this Nov 15, 2017
@GuillaumeGomez
Copy link
Member

I'll add it very soon.

@GuillaumeGomez GuillaumeGomez mentioned this issue Nov 15, 2017
Merged
GuillaumeGomez added a commit to GuillaumeGomez/rust that referenced this issue Nov 16, 2017
@GuillaumeGomez
Rollup merge of rust-lang#46005 - GuillaumeGomez:short-unstable, r=nrc
b09af70 
Set short-message feature unstable

Fixes rust-lang#45995.

r? @nrc
@nikomatsakis
Copy link
Contributor

triage: P-high (I'd prefer to keep my eye on this)

But looks like @GuillaumeGomez is all over it. Thanks @GuillaumeGomez.

@rust-highfive rust-highfive added P-high High priority and removed P-medium Medium priority I-nominated labels Nov 16, 2017
@GuillaumeGomez
Copy link
Member

It's being merged (the PR is complete). I'll give it a high priority.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@GuillaumeGomez GuillaumeGomez

Labels
A-diagnostics Area: Messages for errors, warnings, and lints P-high High priority T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.
Projects
None yet
Milestone
1.23
Development

No branches or pull requests
5 participants
@nikomatsakis @nrc @GuillaumeGomez @Mark-Simulacrum @rust-highfive