Skip to content

Use more concrete explanation for methods #6124

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

Merged
merged 1 commit into from
Oct 6, 2020
Merged

Conversation

pickfire
Copy link
Contributor

@pickfire pickfire commented Oct 6, 2020

Please keep the line below
changelog: none

Show some code rather than "a single method call".
@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 Oct 6, 2020
@Manishearth
Copy link
Member

@bors r+

@bors
Copy link
Contributor

bors commented Oct 6, 2020

📌 Commit 7021d70 has been approved by Manishearth

@bors
Copy link
Contributor

bors commented Oct 6, 2020

⌛ Testing commit 7021d70 with merge 7ab4972...

bors added a commit that referenced this pull request Oct 6, 2020
Use more concrete explanation for methods

*Please keep the line below*
changelog: none
@bors
Copy link
Contributor

bors commented Oct 6, 2020

💔 Test failed - checks-action_test

@flip1995
Copy link
Member

flip1995 commented Oct 6, 2020

@bors r+

@bors
Copy link
Contributor

bors commented Oct 6, 2020

💡 This pull request was already approved, no need to approve it again.

  • This pull request previously failed. You should add more commits to fix the bug, or use retry to trigger a build again.
  • There's another pull request that is currently being tested, blocking this pull request: Sync from rust #6121

@bors
Copy link
Contributor

bors commented Oct 6, 2020

📌 Commit 7021d70 has been approved by flip1995

@bors
Copy link
Contributor

bors commented Oct 6, 2020

⌛ Testing commit 7021d70 with merge c67641f...

bors added a commit that referenced this pull request Oct 6, 2020
Use more concrete explanation for methods

*Please keep the line below*
changelog: none
@bors
Copy link
Contributor

bors commented Oct 6, 2020

💔 Test failed - checks-action_dev_test

@flip1995
Copy link
Member

flip1995 commented Oct 6, 2020

@bors retry (this time remembering, that I don't have to reapprove this!)

@bors
Copy link
Contributor

bors commented Oct 6, 2020

⌛ Testing commit 7021d70 with merge 1bed186...

bors added a commit that referenced this pull request Oct 6, 2020
Use more concrete explanation for methods

*Please keep the line below*
changelog: none
@bors
Copy link
Contributor

bors commented Oct 6, 2020

💔 Test failed - checks-action_test

@flip1995
Copy link
Member

flip1995 commented Oct 6, 2020

@bors retry

@bors
Copy link
Contributor

bors commented Oct 6, 2020

⌛ Testing commit 7021d70 with merge 14e7269...

@bors
Copy link
Contributor

bors commented Oct 6, 2020

☀️ Test successful - checks-action_dev_test, checks-action_remark_test, checks-action_test
Approved by: flip1995
Pushing 14e7269 to master...

@bors bors merged commit 14e7269 into rust-lang:master Oct 6, 2020
@pickfire pickfire deleted the patch-1 branch October 7, 2020 06:15
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.

5 participants