Skip to content

Conversation

@phansch
Copy link
Contributor

@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
Contributor 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
Contributor Author

phansch commented Apr 29, 2019

@bors r+

@bors
Copy link
Contributor

bors commented Apr 29, 2019

📌 Commit 3f637cb has been approved by phansch

@bors
Copy link
Contributor

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
Contributor

bors commented Apr 29, 2019

💔 Test failed - status-appveyor

@phansch
Copy link
Contributor Author

phansch commented Apr 29, 2019

@bors retry (after #4046)

@bors
Copy link
Contributor

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
Contributor

bors commented Apr 29, 2019

📌 Commit 3f637cb has been approved by phansch

@bors
Copy link
Contributor

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
Contributor

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