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

When encountering ||{}(), suggest the likely intended (||{})() #59035

Merged
merged 1 commit into from
Mar 22, 2019

Conversation

estebank
Copy link
Contributor

@estebank estebank commented Mar 9, 2019

Fix #55851.

@rust-highfive
Copy link
Collaborator

r? @davidtwco

(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 Mar 9, 2019
@davidtwco
Copy link
Member

@bors r+

@bors
Copy link
Contributor

bors commented Mar 9, 2019

📌 Commit 9aa89b2 has been approved by davidtwco

@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 Mar 9, 2019
Centril added a commit to Centril/rust that referenced this pull request Mar 10, 2019
…twco

When encountetring `||{}()`, suggest the likely intended `(||{})()`

Fix rust-lang#55851.
Centril added a commit to Centril/rust that referenced this pull request Mar 10, 2019
…twco

When encountetring `||{}()`, suggest the likely intended `(||{})()`

Fix rust-lang#55851.
Centril added a commit to Centril/rust that referenced this pull request Mar 10, 2019
…twco

When encountetring `||{}()`, suggest the likely intended `(||{})()`

Fix rust-lang#55851.
Centril added a commit to Centril/rust that referenced this pull request Mar 11, 2019
…twco

When encountetring `||{}()`, suggest the likely intended `(||{})()`

Fix rust-lang#55851.
pietroalbini added a commit to pietroalbini/rust that referenced this pull request Mar 13, 2019
…twco

When encountetring `||{}()`, suggest the likely intended `(||{})()`

Fix rust-lang#55851.
Centril added a commit to Centril/rust that referenced this pull request Mar 16, 2019
…twco

When encountetring `||{}()`, suggest the likely intended `(||{})()`

Fix rust-lang#55851.
@estebank estebank changed the title When encountetring ||{}(), suggest the likely intended (||{})() When encountering ||{}(), suggest the likely intended (||{})() Mar 18, 2019
@bors
Copy link
Contributor

bors commented Mar 22, 2019

⌛ Testing commit 9aa89b2 with merge 2210e9a...

bors added a commit that referenced this pull request Mar 22, 2019
When encountering `||{}()`, suggest the likely intended `(||{})()`

Fix #55851.
@bors
Copy link
Contributor

bors commented Mar 22, 2019

☀️ Test successful - checks-travis, status-appveyor
Approved by: davidtwco
Pushing 2210e9a to master...

@bors bors added the merged-by-bors This PR was explicitly merged by bors. label Mar 22, 2019
@bors bors merged commit 9aa89b2 into rust-lang:master Mar 22, 2019
@estebank estebank deleted the closure-instacall branch November 9, 2023 05:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
merged-by-bors This PR was explicitly merged by bors. 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.

4 participants