-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Rustup #4388
Rustup #4388
Conversation
try macro is deprecated now, so Clippy will drop the support for it also
Lint redundant_semicolon was added to rustc
Maybe rustup this too: rust-lang/rust#62429 . |
rust-lang/rust#62429 didn't break anything, so we're good here. |
@bors r+ |
📌 Commit 87fa2d9 has been approved by |
Rustup Supersedes #4387 and #4385 This removes tests with the `try!` macro in them completely. There is no need for Clippy to support the `try!` macro, since it is deprecated now. [`StmtKind`](https://doc.rust-lang.org/nightly/nightly-rustc/rustc/hir/enum.StmtKind.html) got a new variant [`Semi`](https://doc.rust-lang.org/nightly/nightly-rustc/rustc/hir/enum.StmtKind.html#variant.Semi), which Just Works™ with the `author` lint. Nice. changelog: none
💔 Test failed - checks-travis |
Unicode breaks the |
@bors retry |
Rustup Supersedes #4387 and #4385 This removes tests with the `try!` macro in them completely. There is no need for Clippy to support the `try!` macro, since it is deprecated now. [`StmtKind`](https://doc.rust-lang.org/nightly/nightly-rustc/rustc/hir/enum.StmtKind.html) got a new variant [`Semi`](https://doc.rust-lang.org/nightly/nightly-rustc/rustc/hir/enum.StmtKind.html#variant.Semi), which Just Works with the `author` lint. Nice. changelog: none
☀️ Test successful - checks-travis, status-appveyor |
Supersedes #4387 and #4385
This removes tests with the
try!
macro in them completely. There is no need for Clippy to support thetry!
macro, since it is deprecated now.StmtKind
got a new variantSemi
, which Just Works with theauthor
lint. Nice.changelog: none