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

Re-enable len_zero for ranges now that is_empty is stable on them #5961

Merged
merged 1 commit into from Aug 25, 2020

Conversation

scottmcm
Copy link
Member

@scottmcm scottmcm commented Aug 25, 2020

Fixes #5956

Completed stabilization PR: rust-lang/rust#75132

changelog: len_zero: re-enable linting ranges now that range_is_empty is stable

@rust-highfive
Copy link

r? @Manishearth

(rust_highfive has picked a reviewer for you, use r? to override)

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label Aug 25, 2020
@Manishearth
Copy link
Member

@bors r+

@bors
Copy link
Collaborator

bors commented Aug 25, 2020

📌 Commit 3b1e5d6 has been approved by Manishearth

@bors
Copy link
Collaborator

bors commented Aug 25, 2020

⌛ Testing commit 3b1e5d6 with merge e191151...

@bors
Copy link
Collaborator

bors commented Aug 25, 2020

☀️ Test successful - checks-action_dev_test, checks-action_remark_test, checks-action_test
Approved by: Manishearth
Pushing e191151 to master...

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.

Reconsider #5656 after Range::is_empty stabilization
4 participants