Skip to content

Reinstate test for forbid(clippy::restriction) #6127

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

Merged
merged 1 commit into from
Oct 6, 2020

Conversation

ebroto
Copy link
Member

@ebroto ebroto commented Oct 6, 2020

In this commit coming from rust-lang/rust (#6126) the test for #![forbid(clippy::restriction)] was removed because some restriction lints were being allowed after the forbid.

This change reinstates the test for forbid by using a separate file.

changelog: none

r? @flip1995

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

flip1995 commented Oct 6, 2020

Thanks! I was too lazy to care about it 😄

@bors r+

@bors
Copy link
Contributor

bors commented Oct 6, 2020

📌 Commit 6c3611b has been approved by flip1995

@bors
Copy link
Contributor

bors commented Oct 6, 2020

⌛ Testing commit 6c3611b with merge c8d89ba...

@bors
Copy link
Contributor

bors commented Oct 6, 2020

☀️ Test successful - checks-action_dev_test, checks-action_remark_test, checks-action_test
Approved by: flip1995
Pushing c8d89ba to master...

@bors bors merged commit c8d89ba into rust-lang:master Oct 6, 2020
@ebroto ebroto deleted the reinstate_forbid_restriction branch October 7, 2020 20:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-review Status: Awaiting review from the assignee but also interested parties
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants