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

Move return_self_not_must_use to pedantic #8302

Merged
merged 1 commit into from
Jan 17, 2022

Conversation

xFrednet
Copy link
Member

r? @flip1995

changelog: none

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

@bors r+ p=1

Thanks!

@bors
Copy link
Contributor

bors commented Jan 17, 2022

📌 Commit 6968382 has been approved by flip1995

bors added a commit that referenced this pull request Jan 17, 2022
…flip1995

Move `return_self_not_must_use` to `pedantic`

r? `@flip1995`

changelog: none
@bors
Copy link
Contributor

bors commented Jan 17, 2022

⌛ Testing commit 6968382 with merge bf2fff3...

@bors
Copy link
Contributor

bors commented Jan 17, 2022

💔 Test failed - checks-action_dev_test

@xFrednet xFrednet force-pushed the 0000-move-return-self-not-must-use branch from 6968382 to 2d3eb50 Compare January 17, 2022 11:35
@xFrednet
Copy link
Member Author

Ups, rustfmt complained

@bors r=flip1995

@bors
Copy link
Contributor

bors commented Jan 17, 2022

📌 Commit 2d3eb50 has been approved by flip1995

@bors
Copy link
Contributor

bors commented Jan 17, 2022

⌛ Testing commit 2d3eb50 with merge 72c59fd...

@bors
Copy link
Contributor

bors commented Jan 17, 2022

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

@bors bors merged commit 72c59fd into rust-lang:master Jan 17, 2022
@xFrednet xFrednet deleted the 0000-move-return-self-not-must-use branch January 17, 2022 11:51
bors added a commit to rust-lang-ci/rust that referenced this pull request Feb 23, 2022
…imulacrum

[stable] 1.59.0 artifacts (second round)

This backports (from 1.60, landed in rust-lang#93001):

*  Move return_self_not_must_use to pedantic rust-lang/rust-clippy#8302

Per a user report on the internals feedback thread, this lint is not behaving well in 1.59.

cc `@rust-lang/clippy` -- this is a stable backport of a patch, which we'll likely want to land in fairly short order to be in time for the release Thursday.

This PR also includes an adjustment to the release notes to reflect "Fix invalid special casing of the unreachable! macro rust-lang#93179".

r? `@Mark-Simulacrum`
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.

5 participants