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.

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

fix help for the --print option #38815

Merged
merged 1 commit into from
Jan 6, 2017
Merged

fix help for the --print option #38815

merged 1 commit into from
Jan 6, 2017

Conversation

cardoe
Copy link
Contributor

@cardoe cardoe commented Jan 4, 2017

Since 8285ab5, which was merged in with #38061, the help for the
--print option is missing the surrounding [ ] around the possible
options.

Signed-off-by: Doug Goldstein cardoe@cardoe.com

Since 8285ab5, which was merged in with rust-lang#38061, the help for the
--print option is missing the surrounding [ ] around the possible
options.

Signed-off-by: Doug Goldstein <cardoe@cardoe.com>
@rust-highfive
Copy link
Collaborator

r? @nikomatsakis

(rust_highfive has picked a reviewer for you, use r? to override)

@nikomatsakis
Copy link
Contributor

@bors r+ rollup

Thanks! :)

@bors
Copy link
Contributor

bors commented Jan 4, 2017

📌 Commit 031dd81 has been approved by nikomatsakis

@cardoe
Copy link
Contributor Author

cardoe commented Jan 4, 2017

@nikomatsakis Not sure if you want to pull this into the beta since this is a UI break.

steveklabnik added a commit to steveklabnik/rust that referenced this pull request Jan 4, 2017
fix help for the --print option

Since 8285ab5, which was merged in with rust-lang#38061, the help for the
--print option is missing the surrounding [ ] around the possible
options.

Signed-off-by: Doug Goldstein <cardoe@cardoe.com>
@nikomatsakis nikomatsakis added the beta-nominated Nominated for backporting to the compiler in the beta channel. label Jan 5, 2017
@nikomatsakis
Copy link
Contributor

Nominated for beta. Seems like a harmless backport. cc @rust-lang/compiler

@cardoe
Copy link
Contributor Author

cardoe commented Jan 5, 2017

FWIW, the output in 1.14.0 is correct. So backporting the fix to beta would cause no user visible regression to occur in a stable release.

@nikomatsakis nikomatsakis added T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. beta-accepted Accepted for backporting to the compiler in the beta channel. labels Jan 6, 2017
@nikomatsakis
Copy link
Contributor

Accepting for beta. Harmless. cc @rust-lang/compiler

@nikomatsakis nikomatsakis mentioned this pull request Jan 6, 2017
17 tasks
@nikomatsakis
Copy link
Contributor

@bors rollup-

@nikomatsakis
Copy link
Contributor

@bors p=1

  • Beta backport

@nikomatsakis nikomatsakis mentioned this pull request Jan 6, 2017
6 tasks
@bors
Copy link
Contributor

bors commented Jan 6, 2017

⌛ Testing commit 031dd81 with merge 373efe8...

bors added a commit that referenced this pull request Jan 6, 2017
fix help for the --print option

Since 8285ab5, which was merged in with #38061, the help for the
--print option is missing the surrounding [ ] around the possible
options.

Signed-off-by: Doug Goldstein <cardoe@cardoe.com>
@bors
Copy link
Contributor

bors commented Jan 6, 2017

☀️ Test successful - status-appveyor, status-travis
Approved by: nikomatsakis
Pushing 373efe8 to master...

@bors bors merged commit 031dd81 into rust-lang:master Jan 6, 2017
@alexcrichton alexcrichton removed the beta-nominated Nominated for backporting to the compiler in the beta channel. label Jan 7, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
beta-accepted Accepted for backporting to the compiler in the beta channel. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants