Skip to content

Conversation

danieledapo
Copy link
Contributor

@danieledapo danieledapo commented Jun 19, 2019

fixes #4181.

changelog: make needless_return work with void functions.

I don't think the failure is related to my changes, but I'm not sure 🤔

@phansch
Copy link
Member

phansch commented Jun 20, 2019

Closing & Re-opening to trigger a new build

@phansch phansch closed this Jun 20, 2019
@phansch phansch reopened this Jun 20, 2019
@flip1995 flip1995 added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label Jun 21, 2019
@flip1995
Copy link
Member

Thanks!

@bors r+

@bors
Copy link
Contributor

bors commented Jun 22, 2019

📌 Commit 316a9f2 has been approved by flip1995

bors added a commit that referenced this pull request Jun 22, 2019
…r=flip1995

make needless_return work with void functions

fixes #4181.

changelog: make needless_return work with void functions.

I don't think the failure is related to my changes, but I'm not sure 🤔
@bors
Copy link
Contributor

bors commented Jun 22, 2019

⌛ Testing commit 316a9f2 with merge 469f73d...

@bors
Copy link
Contributor

bors commented Jun 22, 2019

💔 Test failed - checks-travis

@flip1995
Copy link
Member

waiting for #4227

@flip1995
Copy link
Member

@bors retry

@bors
Copy link
Contributor

bors commented Jun 22, 2019

⌛ Testing commit 316a9f2 with merge c5d1ecd...

bors added a commit that referenced this pull request Jun 22, 2019
…r=flip1995

make needless_return work with void functions

fixes #4181.

changelog: make needless_return work with void functions.

I don't think the failure is related to my changes, but I'm not sure 🤔
@bors
Copy link
Contributor

bors commented Jun 22, 2019

☀️ Test successful - checks-travis, status-appveyor
Approved by: flip1995
Pushing c5d1ecd to master...

@bors bors merged commit 316a9f2 into rust-lang:master Jun 22, 2019
@danieledapo danieledapo deleted the 4182-needless-return-void-functions branch June 22, 2019 10:27
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.

FN needless_return: "return" at end of void-function
4 participants