-
Notifications
You must be signed in to change notification settings - Fork 12.9k
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
Lint on redundant trailing semicolon after item #79812
Conversation
r? @eddyb (rust-highfive has picked a reviewer for you, use r? to override) |
r? @oli-obk |
@bors r+ rollup |
📌 Commit fdc336c has been approved by |
…r=oli-obk Lint on redundant trailing semicolon after item We now lint on code like this: ```rust fn main() { fn foo() {}; struct Bar {}; } ``` Previously, this caused warnings in Cargo, so it was disabled.
fdc336c
to
87c97d3
Compare
@oli-obk: The failure was caused by Rustdoc including an unnecessary semicolon in some of the code that it generates for doctests. This should now be ready to merge. |
Oh, heh, makes sense @bors r+ |
📌 Commit 87c97d3 has been approved by |
…r=oli-obk Lint on redundant trailing semicolon after item We now lint on code like this: ```rust fn main() { fn foo() {}; struct Bar {}; } ``` Previously, this caused warnings in Cargo, so it was disabled.
failed in rollup @bors r- |
We now lint on code like this: ```rust fn main() { fn foo() {}; struct Bar {}; } ``` Previously, this caused warnings in Cargo, so it was disabled.
87c97d3
to
6bef37c
Compare
@bors r=oli-obk |
📌 Commit 6bef37c has been approved by |
…r=oli-obk Lint on redundant trailing semicolon after item We now lint on code like this: ```rust fn main() { fn foo() {}; struct Bar {}; } ``` Previously, this caused warnings in Cargo, so it was disabled.
Rollup of 13 pull requests Successful merges: - rust-lang#79812 (Lint on redundant trailing semicolon after item) - rust-lang#80348 (remove redundant clones (clippy::redundant_clone)) - rust-lang#80358 (Edit rustc_span documentation) - rust-lang#80457 (Add missing commas to `rustc_ast_pretty::pp` docs) - rust-lang#80461 (Add llvm-libunwind change to bootstrap CHANGELOG) - rust-lang#80464 (Use Option::map_or instead of open coding it) - rust-lang#80465 (Fix typo in ffi-pure.md) - rust-lang#80467 (More uses of the matches! macro) - rust-lang#80469 (Fix small typo in time comment) - rust-lang#80472 (Use sans-serif font for the "all items" page links) - rust-lang#80477 (Make forget intrinsic safe) - rust-lang#80482 (don't clone copy types) - rust-lang#80487 (don't redundantly repeat field names) Failed merges: r? `@ghost` `@rustbot` modify labels: rollup
Fix redundant semicolon. Lint introduced by rust-lang/rust#79812.
This will cause a failure due to the warning after the next beta branch as rust-lang#79812 will hit beta.
This will cause a failure due to the warning after the next beta branch as rust-lang#79812 will hit beta.
We now lint on code like this:
Previously, this caused warnings in Cargo, so it was disabled.