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

Package renames should affect fingerprints #5425

Merged
merged 1 commit into from
Apr 28, 2018

Conversation

matklad
Copy link
Member

@matklad matklad commented Apr 27, 2018

No description provided.

@rust-highfive
Copy link

r? @alexcrichton

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

@matklad matklad force-pushed the rename-fingerprint branch 2 times, most recently from 3191d89 to 0f2433c Compare April 28, 2018 10:36
@matklad matklad force-pushed the rename-fingerprint branch from 0f2433c to fc72bcb Compare April 28, 2018 10:38
@matklad matklad changed the title WIP: package renames should affect fingerprints Зackage renames should affect fingerprints Apr 28, 2018
@matklad matklad changed the title Зackage renames should affect fingerprints Package renames should affect fingerprints Apr 28, 2018
@matklad
Copy link
Member Author

matklad commented Apr 28, 2018

Added the actual fix, ready for the review.

@alexcrichton
Copy link
Member

@bors: r+

Awesome, thanks!

@bors
Copy link
Contributor

bors commented Apr 28, 2018

📌 Commit fc72bcb has been approved by alexcrichton

@bors
Copy link
Contributor

bors commented Apr 28, 2018

⌛ Testing commit fc72bcb with merge 2654a6c9ef7fc282c4a230f879cf0a93a007d2b6...

@bors
Copy link
Contributor

bors commented Apr 28, 2018

💔 Test failed - status-travis

@alexcrichton
Copy link
Member

@bors: retry

??

@bors
Copy link
Contributor

bors commented Apr 28, 2018

⌛ Testing commit fc72bcb with merge 450aed598b148eb540ce659818ab474441414be2...

@bors
Copy link
Contributor

bors commented Apr 28, 2018

💔 Test failed - status-travis

@alexcrichton
Copy link
Member

@bors: retry

@bors
Copy link
Contributor

bors commented Apr 28, 2018

⌛ Testing commit fc72bcb with merge 07d022d...

bors added a commit that referenced this pull request Apr 28, 2018
Package renames should affect fingerprints
@bors
Copy link
Contributor

bors commented Apr 28, 2018

☀️ Test successful - status-appveyor, status-travis
Approved by: alexcrichton
Pushing 07d022d to master...

@bors bors merged commit fc72bcb into rust-lang:master Apr 28, 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.

5 participants