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

Record all imports (use, extern crate) in the crate metadata #46419

Merged
merged 2 commits into from
Dec 13, 2017

Conversation

jseyfried
Copy link
Contributor

@jseyfried jseyfried commented Dec 1, 2017

This PR adds non-pub use and extern crate imports in the crate metadata since hygienic macros invoked in other crates may use them. We already include all other non-pub items in the crate metadata. This improves import suggestions in some cases.

Fixes #42337.

r? @nrc

@jseyfried
Copy link
Contributor Author

jseyfried commented Dec 1, 2017

  • Fix fallout in rustdoc.

@kennytm kennytm added the S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. label Dec 1, 2017
@jseyfried jseyfried force-pushed the all_imports_in_metadata branch 4 times, most recently from 3478b62 to 2677909 Compare December 5, 2017 21:13
@jseyfried jseyfried added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. labels Dec 5, 2017
@jseyfried jseyfried force-pushed the all_imports_in_metadata branch from 2677909 to e99c602 Compare December 5, 2017 22:47
@jseyfried jseyfried force-pushed the all_imports_in_metadata branch from e99c602 to 1b9d058 Compare December 6, 2017 01:23
@jseyfried
Copy link
Contributor Author

This is ready for review.

@carols10cents
Copy link
Member

Review ping for you @nrc !

@nrc
Copy link
Member

nrc commented Dec 13, 2017

@bors: r+

@bors
Copy link
Contributor

bors commented Dec 13, 2017

📌 Commit 1b9d058 has been approved by nrc

@bors
Copy link
Contributor

bors commented Dec 13, 2017

⌛ Testing commit 1b9d058 with merge 6110084...

bors added a commit that referenced this pull request Dec 13, 2017
Record all imports (`use`, `extern crate`) in the crate metadata

This PR adds non-`pub` `use` and `extern crate` imports in the crate metadata since hygienic macros invoked in other crates may use them. We already include all other non-`pub` items in the crate metadata. This improves import suggestions in some cases.

Fixes #42337.

r? @nrc
@kennytm kennytm added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Dec 13, 2017
@bors
Copy link
Contributor

bors commented Dec 13, 2017

☀️ Test successful - status-appveyor, status-travis
Approved by: nrc
Pushing 6110084 to master...

@bors bors merged commit 1b9d058 into rust-lang:master Dec 13, 2017
@jseyfried jseyfried deleted the all_imports_in_metadata branch December 13, 2017 18:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants