Skip to content

Attempt to fix nightly manifests - #40111

Merged
bors merged 2 commits into
rust-lang:masterfrom
alexcrichton:fix-manifests
Feb 26, 2017
Merged

bors merged 2 commits into
rust-lang:masterfrom
alexcrichton:fix-manifests

Conversation

@alexcrichton

@alexcrichton alexcrichton commented Feb 26, 2017 •

Copy link
Copy Markdown
Member

Found a few bugs and mistakes with the manifests.

Closes #40106

The name of the package is 'rust-analysis', not 'analysis', and then
name of the channel is 'nightly', not 'rust-nightly'
This was actually just generating invalid hashes for the `rust`
component.
@rust-highfive

Copy link
Copy Markdown
Contributor

r? @nikomatsakis

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

@alexcrichton

Copy link
Copy Markdown
Member Author

@bors: r=brson p=1

@bors

bors commented Feb 26, 2017

Copy link
Copy Markdown
Collaborator

📌 Commit 5ed6765 has been approved by brson

@alexcrichton alexcrichton mentioned this pull request Feb 26, 2017
@bors

bors commented Feb 26, 2017

Copy link
Copy Markdown
Collaborator

⌛ Testing commit 5ed6765 with merge 4baeab6...

@bors

bors commented Feb 26, 2017

Copy link
Copy Markdown
Collaborator

💔 Test failed - status-travis

@ebkalderon

ebkalderon commented Feb 26, 2017 •

Copy link
Copy Markdown
Contributor

Anyone figured out why the checks fail?

@alexcrichton

alexcrichton commented Feb 26, 2017 via email

Copy link
Copy Markdown
Member Author

@bors

bors commented Feb 26, 2017

Copy link
Copy Markdown
Collaborator

⌛ Testing commit 5ed6765 with merge 72e6d40...

bors added a commit that referenced this pull request Feb 26, 2017
Attempt to fix nightly manifests

Found a few bugs and mistakes with the manifests.

Closes #40106
@bors

bors commented Feb 26, 2017

Copy link
Copy Markdown
Collaborator

☀️ Test successful - status-appveyor, status-travis
Approved by: brson
Pushing 72e6d40 to master...

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