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

Beta backport of #40254 #40256

Merged
merged 1 commit into from
Mar 8, 2017
Merged

Conversation

nagisa
Copy link
Member

@nagisa nagisa commented Mar 4, 2017

No description provided.

@rust-highfive
Copy link
Collaborator

r? @alexcrichton

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

@rust-highfive
Copy link
Collaborator

warning Warning warning

  • Pull requests are usually filed against the master branch for this repo, but this one is against beta. Please double check that you specified the right target!

@alexcrichton
Copy link
Member

@bors: r+ p=1

(beta backport)

@bors
Copy link
Contributor

bors commented Mar 6, 2017

📌 Commit e8d90fb has been approved by alexcrichton

@bors
Copy link
Contributor

bors commented Mar 6, 2017

⌛ Testing commit e8d90fb with merge 5047e41...

@bors
Copy link
Contributor

bors commented Mar 6, 2017

💔 Test failed - status-appveyor

@alexcrichton
Copy link
Member

alexcrichton commented Mar 6, 2017 via email

@bors
Copy link
Contributor

bors commented Mar 6, 2017

⌛ Testing commit e8d90fb with merge edf1ef4...

@bors
Copy link
Contributor

bors commented Mar 6, 2017

💔 Test failed - status-travis

@alexcrichton
Copy link
Member

alexcrichton commented Mar 6, 2017 via email

@bors
Copy link
Contributor

bors commented Mar 6, 2017

⌛ Testing commit e8d90fb with merge e204742...

@brson
Copy link
Contributor

brson commented Mar 7, 2017

@alexcrichton Shouldn't this PR include a version bump since every beta publishes? Landing this will publish a beta with the same number as previously.

@bors
Copy link
Contributor

bors commented Mar 7, 2017

💥 Test timed out

@alexcrichton
Copy link
Member

@bors: retry

  • ???

@bors
Copy link
Contributor

bors commented Mar 7, 2017

⌛ Testing commit e8d90fb with merge 6e3a32a...

bors added a commit that referenced this pull request Mar 7, 2017
@nagisa
Copy link
Member Author

nagisa commented Mar 7, 2017

Shouldn't this PR include a version bump since every beta publishes?

So how is it? Seems weird and error prone to make contributors to do this if that’s true.

@alexcrichton
Copy link
Member

Oh sorry I meant to respond to that.

No a version bump isn't necessary. The rust-central-station supports only publishes betas if the version changes, unlike nightlies where a version is published if the revision changes.

@bors
Copy link
Contributor

bors commented Mar 7, 2017

💔 Test failed - status-travis

@alexcrichton
Copy link
Member

alexcrichton commented Mar 7, 2017 via email

@bors
Copy link
Contributor

bors commented Mar 7, 2017

⌛ Testing commit e8d90fb with merge 99b423e...

@bors
Copy link
Contributor

bors commented Mar 7, 2017

💔 Test failed - status-travis

@alexcrichton
Copy link
Member

alexcrichton commented Mar 7, 2017 via email

@bors
Copy link
Contributor

bors commented Mar 8, 2017

⌛ Testing commit e8d90fb with merge 92e7b9b...

bors added a commit that referenced this pull request Mar 8, 2017
@bors
Copy link
Contributor

bors commented Mar 8, 2017

☀️ Test successful - status-appveyor, status-travis
Approved by: alexcrichton
Pushing 92e7b9b to beta...

@bors bors merged commit e8d90fb into rust-lang:beta Mar 8, 2017
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.

5 participants