Stabilize crate_in_paths, extern_absolute_paths and extern_prelude on all editions.#54403
Conversation
This comment has been minimized.
This comment has been minimized.
|
* |
|
@petrochenkov Right, we already have a distinction between "the feature-gate is stable" (which is what this PR affects) and "the feature affects semantics vs migrations" (which this PR doesn't touch). |
85b3d71 to
51abc8a
Compare
|
@bors: r+ |
|
📌 Commit 51abc8a9b8365f961e5a27a071f6d50b64174716 has been approved by |
|
@bors: p=1 |
|
@bors: p=3 |
This comment has been minimized.
This comment has been minimized.
|
@bors: r- |
|
|
51abc8a to
3a29043
Compare
|
@bors r=alexcrichton |
|
📌 Commit 3a290435a872d5a18232272e1947bb9f39fc51d6 has been approved by |
|
It looks like this stabilizes the Edit: nevermind, misread the diff. 😅 |
|
@rpjohnst |
[beta] Stabilize crate_in_paths, extern_absolute_paths and extern_prelude on all editions. Beta backport of #54403. r? @alexcrichton
Remove `fix::local_paths_no_fix`, as `crate_in_paths` is getting stabilized. Needed for rust-lang/rust#54403 (blocking RC1). Ideally we'd also clean up the tests, e.g. removing `#![feature(rust_2018_preview)]` and `is_nightly` checks, but it's easier to just remove the only failing test (because it tests the feature gate is needed).
adc4785 to
0870121
Compare
|
@bors r=alexcrichton |
|
📌 Commit 0870121ed1e97386a1bdfdcf966d2d454eab60d3 has been approved by |
|
The job Click to expand the log.I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
|
This will need all the changes from #54334 if you want to update cargo. |
|
☔ The latest upstream changes (presumably #54334) made this pull request unmergeable. Please resolve the merge conflicts. |
0870121 to
fa2c246
Compare
|
@bors r=alexcrichton |
|
📌 Commit fa2c246 has been approved by |
Stabilize crate_in_paths, extern_absolute_paths and extern_prelude on all editions. Needed for beta, path-related migrations to Rust 2018 don't work on RC1 without these stabilizations. r? @aturon cc @nikomatsakis @Centril @alexcrichton
|
☀️ Test successful - status-appveyor, status-travis |
Stabilization PR: rust-lang/rust#54403
|
A bit late to the punch, but this may have caused a 5-6% decrease in perf across some benchmarks |
Stabilization PR: rust-lang/rust#54403
Needed for beta, path-related migrations to Rust 2018 don't work on RC1 without these stabilizations.
r? @aturon cc @nikomatsakis @Centril @alexcrichton