Skip to content

Add exteranal macros for as_conversions #6317

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
Nov 23, 2020

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