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

Add two more tests for redundant_closure #4032

Merged
merged 1 commit into from Apr 29, 2019
Merged

Conversation

phansch
Copy link
Member

@phansch phansch commented Apr 25, 2019

These two cases were fixed by #4008.

Closes #1439

changelog: none

@matthiaskrgr
Copy link
Member

looks like ui test stdout needs to be update again

These two cases were fixed by rust-lang#4008.

Closes rust-lang#1439

changelog: none
@phansch
Copy link
Member Author

phansch commented Apr 26, 2019

looks like ui test stdout needs to be update again

Woops, fixed 👍

@phansch phansch closed this Apr 28, 2019
@phansch phansch reopened this Apr 28, 2019
@phansch
Copy link
Member Author

phansch commented Apr 29, 2019

@bors r+

@bors
Copy link
Collaborator

bors commented Apr 29, 2019

📌 Commit 3f637cb has been approved by phansch

@bors
Copy link
Collaborator

bors commented Apr 29, 2019

⌛ Testing commit 3f637cb with merge 650e1c9...

bors added a commit that referenced this pull request Apr 29, 2019
Add two more tests for redundant_closure

These two cases were fixed by #4008.

Closes #1439

changelog: none
@bors
Copy link
Collaborator

bors commented Apr 29, 2019

💔 Test failed - status-appveyor

@phansch
Copy link
Member Author

phansch commented Apr 29, 2019

@bors retry (after #4046)

@bors
Copy link
Collaborator

bors commented Apr 29, 2019

⌛ Testing commit 3f637cb with merge d06d3d3...

bors added a commit that referenced this pull request Apr 29, 2019
Add two more tests for redundant_closure

These two cases were fixed by #4008.

Closes #1439

changelog: none
@Manishearth
Copy link
Member

@bors force

@Manishearth Manishearth reopened this Apr 29, 2019
@Manishearth
Copy link
Member

@bors r-

@Manishearth
Copy link
Member

@bors clean force r=phansch

@bors
Copy link
Collaborator

bors commented Apr 29, 2019

📌 Commit 3f637cb has been approved by phansch

@bors
Copy link
Collaborator

bors commented Apr 29, 2019

⌛ Testing commit 3f637cb with merge f1eda09...

bors added a commit that referenced this pull request Apr 29, 2019
Add two more tests for redundant_closure

These two cases were fixed by #4008.

Closes #1439

changelog: none
@bors
Copy link
Collaborator

bors commented Apr 29, 2019

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

@bors bors merged commit 3f637cb into rust-lang:master Apr 29, 2019
@phansch phansch deleted the add_tests branch April 30, 2019 06:11
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.

False positive for redundant_closure lint
4 participants