Skip to content

Conversation

ThibsG
Copy link
Contributor

@ThibsG ThibsG commented Jan 18, 2021

Just a follow up of #6549 that adds a test for this lint.

changelog: none

@rust-highfive
Copy link

r? @flip1995

(rust-highfive has picked a reviewer for you, use r? to override)

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

@bors r+

Thanks! More tests are (almost) always good!

@bors
Copy link
Contributor

bors commented Jan 19, 2021

📌 Commit e33ab3f has been approved by flip1995

@bors
Copy link
Contributor

bors commented Jan 19, 2021

⌛ Testing commit e33ab3f with merge ab1020b...

@bors
Copy link
Contributor

bors commented Jan 19, 2021

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

@bors bors merged commit ab1020b into rust-lang:master Jan 19, 2021
@ThibsG ThibsG deleted the AddTestNeedlessReturn branch January 24, 2021 11:14
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.

4 participants