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.

Sign up for GitHub

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

Jump to bottom

fix: switch to inclusive range #11

Merged
merged 2 commits into from
Oct 9, 2019
Merged

fix: switch to inclusive range #11

merged 2 commits into from
Oct 9, 2019

Conversation

trivikr
Copy link
Contributor

@trivikr trivikr commented Oct 8, 2019

Fixes: #10

@tromey
Copy link
Owner

tromey commented Oct 8, 2019

I think over in #10 we agreed to just use ..= and drop backward compatibility. Could you do that instead?

@trivikr trivikr changed the title fix: disable range patterns warning fix: switch to inclusive range Oct 9, 2019
@trivikr
Copy link
Contributor Author

trivikr commented Oct 9, 2019

Could you do that instead?

Done 👍

@tromey tromey merged commit cb192ac into tromey:master Oct 9, 2019
@tromey
Copy link
Owner

tromey commented Oct 9, 2019

Thank you.

@trivikr trivikr deleted the allow_ellipsis_inclusive_range_patterns branch October 9, 2019 14:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

rustc warning: ... range patterns are deprecated
2 participants
@trivikr @tromey