Skip to content

switch to using the --edition flag #5410

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
Apr 27, 2018
Merged

Conversation

klnusbaum
Copy link
Contributor

Now that we have an --edition flag in the rust compiler, let's switch to using it.

fixes #5406

@rust-highfive
Copy link

r? @matklad

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

@matklad
Copy link
Member

matklad commented Apr 23, 2018

Thanks! We’ve tried it some, and at that time there were no nightly with this flag. Let’s try again?

@bors r+

@bors
Copy link
Contributor

bors commented Apr 23, 2018

📌 Commit 0b2aaaf has been approved by matklad

@bors
Copy link
Contributor

bors commented Apr 23, 2018

⌛ Testing commit 0b2aaaf with merge 3679589d1b1aef703935052c2e9dbfb57bdb1afc...

@bors
Copy link
Contributor

bors commented Apr 23, 2018

💔 Test failed - status-appveyor

@dwijnand
Copy link
Member

same failure as my earlier PR #5408

@matklad
Copy link
Member

matklad commented Apr 27, 2018

@bors r+

@bors
Copy link
Contributor

bors commented Apr 27, 2018

💡 This pull request was already approved, no need to approve it again.

  • This pull request previously failed. You should add more commits to fix the bug, or use retry to trigger a build again.

@bors
Copy link
Contributor

bors commented Apr 27, 2018

📌 Commit 0b2aaaf has been approved by matklad

@bors
Copy link
Contributor

bors commented Apr 27, 2018

⌛ Testing commit 0b2aaaf with merge 9eece36...

bors added a commit that referenced this pull request Apr 27, 2018
switch to using the --edition flag

Now that we have an `--edition` flag in the rust compiler, let's switch to using it.

fixes #5406
@bors
Copy link
Contributor

bors commented Apr 27, 2018

☀️ Test successful - status-appveyor, status-travis
Approved by: matklad
Pushing 9eece36 to master...

@bors bors merged commit 0b2aaaf into rust-lang:master Apr 27, 2018
@ehuss ehuss added this to the 1.27.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.

Switch to using --edition
6 participants