Skip to content

Fix ellipsis_inclusive_range_patterns lint warnings#4158

Merged
bors merged 1 commit into
rust-lang:masterfrom
phansch:rustup_tests
Jun 1, 2019
Merged

Fix ellipsis_inclusive_range_patterns lint warnings#4158
bors merged 1 commit into
rust-lang:masterfrom
phansch:rustup_tests

Conversation

@phansch

@phansch phansch commented Jun 1, 2019

Copy link
Copy Markdown
Contributor

Changed from allow to warn in rust-lang/rust#61342

changelog: none

@Manishearth

Copy link
Copy Markdown
Member

@bors r+

@bors

bors commented Jun 1, 2019

Copy link
Copy Markdown
Contributor

📌 Commit 6feef17 has been approved by Manishearth

@bors

bors commented Jun 1, 2019

Copy link
Copy Markdown
Contributor

⌛ Testing commit 6feef17 with merge f3087c3...

bors added a commit that referenced this pull request Jun 1, 2019
Fix ellipsis_inclusive_range_patterns lint warnings

Changed from `allow` to `warn` in rust-lang/rust#61342

changelog: none
@bors

bors commented Jun 1, 2019

Copy link
Copy Markdown
Contributor

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

@bors bors merged commit 6feef17 into rust-lang:master Jun 1, 2019
@phansch

phansch commented Jun 1, 2019

Copy link
Copy Markdown
Contributor Author

The dogfood tests now seem to take at least twice as long on my machine (Xubuntu 18.04, i7-8550U), but I'm not sure why. It looks like this may introduce spurious failures on travis due to the 10 minute timeout. It was already hit here: https://travis-ci.com/rust-lang/rust-clippy/jobs/204682946#L1312

@phansch phansch deleted the rustup_tests branch June 1, 2019 06:41
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.

3 participants