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

Add semicolon to "perhaps add a use for one of them" help #37759

Merged
merged 1 commit into from
Nov 17, 2016

Conversation

robinst
Copy link
Contributor

@robinst robinst commented Nov 14, 2016

Similar to pull request #37430, this makes the message more copy-paste
friendly and aligns it with other messages like:

help: you can import it into scope: use foo::Bar;

r? @eddyb

Similar to pull request rust-lang#37430, this makes the message more copy-paste
friendly and aligns it with other messages like:

    help: you can import it into scope: use foo::Bar;
@eddyb
Copy link
Member

eddyb commented Nov 14, 2016

@bors r+ rollup

@bors
Copy link
Contributor

bors commented Nov 14, 2016

📌 Commit f3a71d3 has been approved by eddyb

GuillaumeGomez added a commit to GuillaumeGomez/rust that referenced this pull request Nov 17, 2016
…olon, r=eddyb

Add semicolon to "perhaps add a `use` for one of them" help

Similar to pull request rust-lang#37430, this makes the message more copy-paste
friendly and aligns it with other messages like:

    help: you can import it into scope: use foo::Bar;

r? @eddyb
GuillaumeGomez added a commit to GuillaumeGomez/rust that referenced this pull request Nov 17, 2016
…olon, r=eddyb

Add semicolon to "perhaps add a `use` for one of them" help

Similar to pull request rust-lang#37430, this makes the message more copy-paste
friendly and aligns it with other messages like:

    help: you can import it into scope: use foo::Bar;

r? @eddyb
bors added a commit that referenced this pull request Nov 17, 2016
Rollup of 8 pull requests

- Successful merges: #37752, #37757, #37759, #37766, #37772, #37799, #37806, #37821
- Failed merges: #37442
@bors bors merged commit f3a71d3 into rust-lang:master Nov 17, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants