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 Clippy ICE fix to beta #65305

Merged
merged 1 commit into from
Oct 15, 2019

Conversation

flip1995
Copy link
Member

There was an ICE in Clippy, when compiling mysql_common. Since this is a pretty popular crate, and there were a bunch of Clippy issues in the last few days, I think it is worth it to backport the fix (rust-lang/rust-clippy#4590) to beta.

cc rust-lang/rust-clippy#4653

r? @Manishearth

@rust-highfive
Copy link
Collaborator

⚠️ 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!
  • These commits modify submodules.

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Oct 11, 2019
@flip1995
Copy link
Member Author

flip1995 commented Oct 11, 2019

Clippy branch for the backport is https://github.com/rust-lang/rust-clippy/tree/rust-1.39.0

@Centril
Copy link
Contributor

Centril commented Oct 11, 2019

r? @pietroalbini

@pietroalbini
Copy link
Member

The backport itself looks fine (just a note: rename the branch from 1.39.0-beta to rust-1.39.0.

@rust-lang/clippy do y'all approve this backport? Not sure how the approval process looks like in your team.

@pietroalbini pietroalbini added S-waiting-on-team Status: Awaiting decision from the relevant subteam (see the T-<team> label). T-dev-tools Relevant to the dev-tools subteam, which will review and decide on the PR/issue. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Oct 11, 2019
@flip1995
Copy link
Member Author

I renamed the branch. 👍

@Manishearth
Copy link
Member

r+

@pietroalbini
Copy link
Member

@bors r+

@bors
Copy link
Contributor

bors commented Oct 11, 2019

📌 Commit 8aacf98 has been approved by pietroalbini

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-team Status: Awaiting decision from the relevant subteam (see the T-<team> label). labels Oct 11, 2019
@bors
Copy link
Contributor

bors commented Oct 15, 2019

⌛ Testing commit 8aacf98 with merge 224f0bc...

bors added a commit that referenced this pull request Oct 15, 2019
[Beta] Backport Clippy ICE fix to beta

There was an ICE in Clippy, when compiling `mysql_common`. Since this is a pretty popular crate, and there were a bunch of Clippy issues in the last few days, I think it is worth it to backport the fix (rust-lang/rust-clippy#4590) to beta.

cc rust-lang/rust-clippy#4653

r? @Manishearth
@bors
Copy link
Contributor

bors commented Oct 15, 2019

☀️ Test successful - checks-azure
Approved by: pietroalbini
Pushing 224f0bc to beta...

@bors bors added the merged-by-bors This PR was explicitly merged by bors. label Oct 15, 2019
@bors bors merged commit 8aacf98 into rust-lang:beta Oct 15, 2019
@flip1995 flip1995 deleted the clippy-beta-backport branch October 15, 2019 10:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
merged-by-bors This PR was explicitly merged by bors. S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. T-dev-tools Relevant to the dev-tools subteam, which will review and decide on the PR/issue.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants