Skip to content

Conversation

0xPoe
Copy link
Member

@0xPoe 0xPoe commented Jun 25, 2021

@rust-highfive
Copy link

r? @Eh2406

(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 25, 2021
@Eh2406
Copy link
Contributor

Eh2406 commented Jun 25, 2021

Should this get a test?

@0xPoe
Copy link
Member Author

0xPoe commented Jun 25, 2021

Should this get a test?

Yes, I think it needs a test. But no relevant test example was found. How do you suggest to test it?

@ehuss
Copy link
Contributor

ehuss commented Jun 25, 2021

Thanks! A test would be nice, but it would be quite difficult to test this reliably (it would need to override the sysroot with a custom one with rust-src missing). I think it is fine to go without for now.

@bors r+

@bors
Copy link
Contributor

bors commented Jun 25, 2021

📌 Commit 00bf770 has been approved by ehuss

@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 Jun 25, 2021
@bors
Copy link
Contributor

bors commented Jun 25, 2021

⌛ Testing commit 00bf770 with merge 3691752...

@bors
Copy link
Contributor

bors commented Jun 25, 2021

☀️ Test successful - checks-actions
Approved by: ehuss
Pushing 3691752 to master...

@bors bors merged commit 3691752 into rust-lang:master Jun 25, 2021
@0xPoe 0xPoe deleted the rustin-msg branch June 26, 2021 02:44
bors added a commit to rust-lang-ci/rust that referenced this pull request Jul 1, 2021
Update cargo

9 commits in 9233aa06c801801cff75df65df718d70905a235e..4952979031e2cf1d901c817a32e25a156a19db4c
2021-06-22 21:32:55 +0000 to 2021-07-01 01:14:50 +0000
- Fix `BorrowMutError` when calling `cargo doc --open` (rust-lang/cargo#9531)
- Exclude `target` from content-indexing on Windows (rust-lang/cargo#9635)
- Temporarily ignore 2021 edition fix. (rust-lang/cargo#9642)
- Temporarily disable future_incompat tests. (rust-lang/cargo#9638)
- Include toolchain specification in error message (rust-lang/cargo#9625)
- Error when packaging with git dependencies without version (rust-lang/cargo#9612)
- simply 'if' block (rust-lang/cargo#9615)
- tidy some closures and iterators (rust-lang/cargo#9614)
- use 'writeln' instead of appending newline character (rust-lang/cargo#9620)
@ehuss ehuss added this to the 1.55.0 milestone Feb 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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.

5 participants