Skip to content

extern crate changes. - #455

Merged
matthewjasper merged 2 commits into
rust-lang:masterfrom
ehuss:extern-crate
Nov 1, 2018
Merged

extern crate changes.#455
matthewjasper merged 2 commits into
rust-lang:masterfrom
ehuss:extern-crate

Conversation

@ehuss

Copy link
Copy Markdown
Contributor

Stabilization PR: rust-lang/rust#54403

Comment threadsrc/items/extern-crates.md Outdated
Possible upcoming changes that will change this:

`extern crate` items will automatically be added to the extern prelude.
https://github.com/rust-lang/rust/pull/54658

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is merged now

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The changes are behind the extern_crate_item_prelude feature flag, so there's not much difference. I don't know what the future holds, there are no tracking issues. I tweaked the wording slightly, but it still seems a little awkward to me. Is the distinction between the --extern added to extern prelude and extern crate being placed as an item in the current scope clear enough? Should it emphasize this distinction more?

@matthewjaspermatthewjasper left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm happy to merge this once the conflict is resolved

@ehuss

ehuss commented Nov 1, 2018

Copy link
Copy Markdown
ContributorAuthor

I'm happy to merge this once the conflict is resolved

Oops, I'm so used to bors leaving a message I forget to check.

@matthewjasper
matthewjasper merged commit 803412c into rust-lang:masterNov 1, 2018
Sign up for freeto 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.

2 participants

@ehuss@matthewjasper