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

Rustup #4388

Merged
merged 3 commits into from
Aug 15, 2019
Merged

Rustup #4388

merged 3 commits into from
Aug 15, 2019

Conversation

flip1995
Copy link
Member

@flip1995 flip1995 commented Aug 15, 2019

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 got a new variant Semi, which Just Works with the author lint. Nice.

changelog: none

try macro is deprecated now, so Clippy will drop the support for it also
Lint redundant_semicolon was added to rustc
@tesuji
Copy link
Contributor

tesuji commented Aug 15, 2019

Maybe rustup this too: rust-lang/rust#62429 .

@flip1995
Copy link
Member Author

rust-lang/rust#62429 didn't break anything, so we're good here.

@phansch
Copy link
Member

phansch commented Aug 15, 2019

@bors r+

@bors
Copy link
Collaborator

bors commented Aug 15, 2019

📌 Commit 87fa2d9 has been approved by phansch

@bors
Copy link
Collaborator

bors commented Aug 15, 2019

⌛ Testing commit 87fa2d9 with merge 3fc4975...

bors added a commit that referenced this pull request Aug 15, 2019
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
@bors
Copy link
Collaborator

bors commented Aug 15, 2019

💔 Test failed - checks-travis

@flip1995
Copy link
Member Author

UnicodeEncodeError: 'ascii' codec can't encode character u'\u2122' in position 406: ordinal not in range(128)

Unicode breaks the changelog check 😄

@flip1995
Copy link
Member Author

@bors retry

@bors
Copy link
Collaborator

bors commented Aug 15, 2019

⌛ Testing commit 87fa2d9 with merge 34457fb...

bors added a commit that referenced this pull request Aug 15, 2019
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
@bors
Copy link
Collaborator

bors commented Aug 15, 2019

☀️ Test successful - checks-travis, status-appveyor
Approved by: phansch
Pushing 34457fb to master...

@bors bors merged commit 87fa2d9 into rust-lang:master Aug 15, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants