Skip to content

No longer warn about #![feature(rust_2018_preview)] #6052

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

Merged
merged 1 commit into from
Sep 18, 2018

Conversation

steveklabnik
Copy link
Member

This is no longer needed, and so the warning is outdated.

Fixes #6050

@rust-highfive
Copy link

r? @matklad

(rust_highfive has picked a reviewer for you, use r? to override)

@joshtriplett
Copy link
Member

@bors r+

@bors
Copy link
Contributor

bors commented Sep 18, 2018

📌 Commit 7980cc9e14c8e5bd55637a835cb5c8a6e5930f29 has been approved by joshtriplett

@bors
Copy link
Contributor

bors commented Sep 18, 2018

⌛ Testing commit 7980cc9e14c8e5bd55637a835cb5c8a6e5930f29 with merge a48fa3df9c07994cb9f8516cd49a4ea93cc0c014...

@aturon
Copy link
Member

aturon commented Sep 18, 2018

cc @alexcrichton, just to make sure everyone's in the loop.

@bors
Copy link
Contributor

bors commented Sep 18, 2018

💔 Test failed - status-travis

This is no longer needed, and so the warning is outdated.

Fixes rust-lang#6050
@steveklabnik
Copy link
Member Author

@bors: r=joshtriplett

Some of the things being tested here are not yet stable, and so do require the edition to fix. As such, they still need the flag in order to pass the tests. I've fixed the tests now; I didn't see that they only run on nightly because they report themselves as passing in other cases.

@bors
Copy link
Contributor

bors commented Sep 18, 2018

📌 Commit d277c84 has been approved by joshtriplett

bors added a commit that referenced this pull request Sep 18, 2018
No longer warn about #![feature(rust_2018_preview)]

This is no longer needed, and so the warning is outdated.

Fixes #6050
@bors
Copy link
Contributor

bors commented Sep 18, 2018

⌛ Testing commit d277c84 with merge 93e3216...

@ehuss
Copy link
Contributor

ehuss commented Sep 18, 2018

I'm still a little confused about this. Does cargo fix --edition do anything without rust_2018_preview? If you don't include that feature, then the new experience will be:

  1. Run cargo fix --edition to transition to the new edition.
  2. Edit Cargo.toml to add edition="2018"
  3. Run cargo build, and it will fail because nothing was fixed.

How is this supposed to work?

@steveklabnik
Copy link
Member Author

Does cargo fix --edition do anything without rust_2018_preview?

Yes.

> cat .\src\lib.rs
trait Foo {
    fn foo(&self, u8);
}
> cargo +nightly fix --edition
    Checking editiontest v0.1.0 (C:\Users\steve\tmp\editiontest)
warning: failed to find `#![feature(rust_2018_preview)]` in `src\lib.rs`
this may cause `cargo fix` to not be able to fix all
issues in preparation for the 2018 edition
      Fixing src\lib.rs (1 fix)
    Finished dev [unoptimized + debuginfo] target(s) in 0.64s
> cat .\src\lib.rs
trait Foo {
    fn foo(&self, _: u8);
}

@bors
Copy link
Contributor

bors commented Sep 18, 2018

☀️ Test successful - status-appveyor, status-travis
Approved by: joshtriplett
Pushing 93e3216 to master...

@bors bors merged commit d277c84 into rust-lang:master Sep 18, 2018
steveklabnik added a commit to steveklabnik/rust that referenced this pull request Sep 18, 2018
This pulls in rust-lang/cargo#6052, which is needed for the edition preview.
bors added a commit that referenced this pull request Sep 19, 2018
No longer warn about #![feature(rust_2018_preview)]

Backport of #6052 to 1.30
bors added a commit to rust-lang/rust that referenced this pull request Sep 22, 2018
update cargo

This pulls in rust-lang/cargo#6052, which is needed for the edition preview, and will be backported to beta as well.
@ehuss ehuss added this to the 1.31.0 milestone Feb 6, 2022
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.

cargo fix still warns about rust_2018_preview
7 participants