Skip to content

Conversation

@ebroto
Copy link
Contributor

@ebroto ebroto commented Jun 5, 2020

It seems we hit rust-lang/cargo#5418, so I've applied the suggested solution. Also added some more info when cargo-metadata fails to execute.

(there was no open issue for this)

changelog: none

@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 Jun 5, 2020
@ebroto
Copy link
Contributor Author

ebroto commented Jun 5, 2020

r? @flip1995 as they are familiar with the issue

@rust-highfive rust-highfive assigned flip1995 and unassigned Manishearth Jun 5, 2020
@flip1995
Copy link
Member

flip1995 commented Jun 5, 2020

Thanks! I tested it in the rustc codebase and it worked!

@bors r+

@bors
Copy link
Contributor

bors commented Jun 5, 2020

📌 Commit c325c12 has been approved by flip1995

@bors
Copy link
Contributor

bors commented Jun 5, 2020

⌛ Testing commit c325c12 with merge ea7066a...

@bors
Copy link
Contributor

bors commented Jun 5, 2020

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

@bors bors merged commit ea7066a into rust-lang:master Jun 5, 2020
@ebroto ebroto deleted the fix_cargo_tests_in_rustc branch June 5, 2020 22:28
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