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

Stabilize pattern_parentheses feature #54497

Merged
merged 2 commits into from
Sep 26, 2018

Conversation

ralexstokes
Copy link
Contributor

Addresses #51087 .

Stabilizes the previously unstable feature pattern_parentheses which enables the use of () in match patterns.

@rust-highfive
Copy link
Collaborator

r? @nikomatsakis

(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 Sep 23, 2018
BitXor | BitAnd | BitOr | Shl | Shr =>
false,
Eq | Lt | Le | Ne | Gt | Ge => true,
And | Or | Add | Sub | Mul | Div | Rem | BitXor | BitAnd | BitOr | Shl | Shr => false,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nit: It's not super important, but it's always nice to factor out rustfmt runs into their own commit (it'd be nice if we just rustfmt'd everything ... but that's neither here nor there)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

whoops -- yeah this was a mistake on my part -- i was only going to make this change with the typo: de5f6ee#diff-0de6e63ecfba5c83fb06c9e981f292a6R636

and not rustfmt this file but i guess it slipped through.

@nikomatsakis
Copy link
Contributor

Looks good!

@nikomatsakis
Copy link
Contributor

@bors r+

@bors
Copy link
Contributor

bors commented Sep 25, 2018

📌 Commit a381868 has been approved by nikomatsakis

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Sep 25, 2018
@Centril Centril added the relnotes Marks issues that should be documented in the release notes of the next release. label Sep 25, 2018
@bors
Copy link
Contributor

bors commented Sep 26, 2018

⌛ Testing commit a381868 with merge a2b27c1...

bors added a commit that referenced this pull request Sep 26, 2018
…ikomatsakis

Stabilize pattern_parentheses feature

Addresses #51087 .

Stabilizes the previously unstable feature `pattern_parentheses` which enables the use of `()` in match patterns.
@bors
Copy link
Contributor

bors commented Sep 26, 2018

☀️ Test successful - status-appveyor, status-travis
Approved by: nikomatsakis
Pushing a2b27c1 to master...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
relnotes Marks issues that should be documented in the release notes of the next release. S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants