Skip to content

Update clippy - #46937

Closed
Manishearth wants to merge 1 commit into
rust-lang:masterfrom
Manishearth:fx-clip
Closed

Update clippy#46937
Manishearth wants to merge 1 commit into
rust-lang:masterfrom
Manishearth:fx-clip

Conversation

@Manishearth

Copy link
Copy Markdown
Member

@rust-highfive

Copy link
Copy Markdown
Contributor

r? @aturon

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

@Manishearth

Copy link
Copy Markdown
MemberAuthor

r? @oli-obk

@oli-obk

Copy link
Copy Markdown
Contributor

You need to run x.py to update the Cargo.lock

@Manishearth

Copy link
Copy Markdown
MemberAuthor

pushed

@kennytm

Copy link
Copy Markdown
Member

@bors r+

@bors

bors commented Dec 22, 2017

Copy link
Copy Markdown
Collaborator

📌 Commit 5935f1f has been approved by kennytm

@kennytmkennytm added the S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. label Dec 22, 2017
@kennytmkennytm mentioned this pull request Dec 23, 2017
kennytm added a commit to kennytm/rust that referenced this pull request Dec 23, 2017
@bors

bors commented Dec 25, 2017

Copy link
Copy Markdown
Collaborator

🔒 Merge conflict

@kennytmkennytm added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. and removed S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. labels Dec 25, 2017
@kennytm

Copy link
Copy Markdown
Member

@Manishearth Would you mind reviving this (rebase on rust master + update clippy to latest master)? Clippy (and miri as well) needs to be test-pass before branching off to the next beta.

@oli-obk

Copy link
Copy Markdown
Contributor

Clippy and miri aren't distributed, why is the beta blocked on them passing?

@kennytm

Copy link
Copy Markdown
Member

@oli-obk it is tested in the same manner as RLS and rustfmt.

Anyway, this is unblocked on 1.24-beta by not testing them.

@kennytm

Copy link
Copy Markdown
Member

Superseded by #47172.

@kennytmkennytm closed this Jan 4, 2018
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-authorStatus: This is awaiting some action (such as code changes or more information) from the author.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants

@Manishearth@rust-highfive@oli-obk@kennytm@bors@aturon