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

Change span of unnecessary_fold lint #4382

Merged
merged 4 commits into from
Aug 15, 2019

Conversation

jeremystucki
Copy link
Contributor

Resolves #4381

changelog: Change linted span of unnecessary_fold

@flip1995
Copy link
Member

Wow that was fast, and the code even got simpler. Thanks!

@bors r+

@bors
Copy link
Collaborator

bors commented Aug 14, 2019

📌 Commit fdf82eb has been approved by flip1995

@bors
Copy link
Collaborator

bors commented Aug 14, 2019

⌛ Testing commit fdf82eb with merge 92df768...

bors added a commit that referenced this pull request Aug 14, 2019
Change span of unnecessary_fold lint

Resolves #4381

changelog: Change linted span of `unnecessary_fold`
@bors
Copy link
Collaborator

bors commented Aug 14, 2019

💔 Test failed - checks-travis

@matthiaskrgr
Copy link
Member

@bors retry spurious failure

@bors
Copy link
Collaborator

bors commented Aug 14, 2019

⌛ Testing commit fdf82eb with merge bcd0e44...

bors added a commit that referenced this pull request Aug 14, 2019
Change span of unnecessary_fold lint

Resolves #4381

changelog: Change linted span of `unnecessary_fold`
@bors
Copy link
Collaborator

bors commented Aug 14, 2019

💔 Test failed - checks-travis

@flip1995
Copy link
Member

@bors retry

@bors
Copy link
Collaborator

bors commented Aug 15, 2019

⌛ Testing commit fdf82eb with merge 8f4b5c8...

bors added a commit that referenced this pull request Aug 15, 2019
Change span of unnecessary_fold lint

Resolves #4381

changelog: Change linted span of `unnecessary_fold`
@bors
Copy link
Collaborator

bors commented Aug 15, 2019

💔 Test failed - status-appveyor

@flip1995
Copy link
Member

@bors retry

@bors
Copy link
Collaborator

bors commented Aug 15, 2019

⌛ Testing commit fdf82eb with merge 525e202...

bors added a commit that referenced this pull request Aug 15, 2019
Change span of unnecessary_fold lint

Resolves #4381

changelog: Change linted span of `unnecessary_fold`
@bors
Copy link
Collaborator

bors commented Aug 15, 2019

💔 Test failed - checks-travis

@phansch
Copy link
Member

phansch commented Aug 15, 2019

@bors retry (spurious network error)

@bors
Copy link
Collaborator

bors commented Aug 15, 2019

⌛ Testing commit fdf82eb with merge 607b829...

bors added a commit that referenced this pull request Aug 15, 2019
Change span of unnecessary_fold lint

Resolves #4381

changelog: Change linted span of `unnecessary_fold`
@bors
Copy link
Collaborator

bors commented Aug 15, 2019

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

@bors bors merged commit fdf82eb into rust-lang:master Aug 15, 2019
@jeremystucki jeremystucki deleted the unnecessary_fold_span branch August 15, 2019 17:27
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.

unnecessary_fold span could be improved
5 participants