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

Improve the warning cycle for use $crate; #39077

Merged
merged 1 commit into from
Jan 21, 2017

Conversation

jseyfried
Copy link
Contributor

Fixes #39049.
r? @nrc

@jseyfried jseyfried added beta-nominated Nominated 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. labels Jan 15, 2017
@nrc
Copy link
Member

nrc commented Jan 17, 2017

@bors: r+

@bors
Copy link
Contributor

bors commented Jan 17, 2017

📌 Commit c02d577 has been approved by nrc

@brson brson added the beta-accepted Accepted for backporting to the compiler in the beta channel. label Jan 17, 2017
@brson
Copy link
Contributor

brson commented Jan 17, 2017

Accepting for backport assuming it cherry-picks.

@brson brson mentioned this pull request Jan 17, 2017
@alexcrichton alexcrichton removed the beta-nominated Nominated for backporting to the compiler in the beta channel. label Jan 17, 2017
@jseyfried jseyfried mentioned this pull request Jan 17, 2017
@bors
Copy link
Contributor

bors commented Jan 20, 2017

⌛ Testing commit c02d577 with merge 58293d8...

alexcrichton added a commit to alexcrichton/rust that referenced this pull request Jan 20, 2017
Improve the warning cycle for `use $crate;`

Fixes rust-lang#39049.
r? @nrc
@alexcrichton
Copy link
Member

@bors: retry

  • prioritizing rollup

bors added a commit that referenced this pull request Jan 20, 2017
bors added a commit that referenced this pull request Jan 21, 2017
@bors bors merged commit c02d577 into rust-lang:master Jan 21, 2017
@bors
Copy link
Contributor

bors commented Jan 21, 2017

⌛ Testing commit c02d577 with merge aedb49c...

@jseyfried jseyfried deleted the crate_var_imports branch January 24, 2017 12:21
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