Skip to content

Rollup of 2 pull requests #3584

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

Closed
wants to merge 6 commits into from
Closed

Rollup of 2 pull requests #3584

wants to merge 6 commits into from

Conversation

flip1995
Copy link
Member

Successful merges:

Failed merges:

r? @ghost

daxpedda and others added 6 commits December 24, 2018 22:06
Co-Authored-By: daxpedda <1645124+daxpedda@users.noreply.github.com>
Fix false positives for `implicit_return` and `empty_loop` on macro expansion.

This PR only fixes `implicit_return` and `empty_loop`.
But I suspect this bug may affect a lot of other lints.
update CARGO_CLIPPY_HELP string to suggest tool lints.
@flip1995
Copy link
Member Author

@bors r+

@bors
Copy link
Contributor

bors commented Dec 26, 2018

📌 Commit 0a10f4a has been approved by flip1995

bors added a commit that referenced this pull request Dec 26, 2018
Rollup of 2 pull requests

Successful merges:

 - #3577 (Fix false positives for `implicit_return` and `empty_loop` on macro expansion.)
 - #3579 (update CARGO_CLIPPY_HELP string to suggest tool lints.)

Failed merges:

r? @ghost
@bors
Copy link
Contributor

bors commented Dec 26, 2018

⌛ Testing commit 0a10f4a with merge 90b673b...

@bors
Copy link
Contributor

bors commented Dec 26, 2018

💔 Test failed - status-travis

@flip1995 flip1995 closed this Dec 26, 2018
@flip1995 flip1995 deleted the rollup branch December 26, 2018 17:10
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