Skip to content

Conversation

chansuke
Copy link
Contributor

@chansuke chansuke commented Nov 10, 2020

Added external macros to this PR #4888.

r? @llogiq

changelog: none

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label Nov 10, 2020
@llogiq
Copy link
Contributor

llogiq commented Nov 21, 2020

Thanks for fixing this oversight. @bors r+

@bors
Copy link
Contributor

bors commented Nov 21, 2020

📌 Commit faa3e23 has been approved by llogiq

@bors
Copy link
Contributor

bors commented Nov 21, 2020

⌛ Testing commit faa3e23 with merge e23b087...

bors added a commit that referenced this pull request Nov 21, 2020
Add exteranal macros for as_conversions

Added external macros to this PR #4888.

r? `@llogiq`
@bors
Copy link
Contributor

bors commented Nov 21, 2020

💔 Test failed - checks-action_test

@llogiq
Copy link
Contributor

llogiq commented Nov 22, 2020

Oops, this appears to need a rustup, which should have been introduced in the meantime. Can you rebase?

@flip1995
Copy link
Member

@bors retry (If no conflicts after a rustup occur, no rebase should be necessary)

@bors
Copy link
Contributor

bors commented Nov 23, 2020

⌛ Testing commit faa3e23 with merge eaeb3c0...

bors added a commit that referenced this pull request Nov 23, 2020
Add exteranal macros for as_conversions

Added external macros to this PR #4888.

r? `@llogiq`
@bors
Copy link
Contributor

bors commented Nov 23, 2020

💔 Test failed - checks-action_test

@flip1995
Copy link
Member

@bors retry (changelog was missing)

@bors
Copy link
Contributor

bors commented Nov 23, 2020

⌛ Testing commit faa3e23 with merge 723ac0f...

@bors
Copy link
Contributor

bors commented Nov 23, 2020

☀️ Test successful - checks-action_dev_test, checks-action_remark_test, checks-action_test
Approved by: llogiq
Pushing 723ac0f to master...

@bors bors merged commit 723ac0f into rust-lang:master Nov 23, 2020
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