Skip to content

Conversation

@phansch
Copy link
Contributor

@phansch phansch commented Feb 15, 2021

These updates allow us to specify multiple testnames for TESTNAME by
providing a comma separated list of testnames.

The new version of compiletest-rs also includes bless support, but is
not enabled with this PR.

cc #5394

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 Feb 15, 2021
@phansch
Copy link
Contributor Author

phansch commented Feb 15, 2021

r? @flip1995

These updates allow us to specify multiple testnames for `TESTNAME`.

The new version of compiletest-rs also includes `bless` support, but is
not enabled with this PR.
@Manishearth
Copy link
Member

@bors r+

@bors
Copy link
Contributor

bors commented Feb 15, 2021

📌 Commit 7226291 has been approved by Manishearth

@bors
Copy link
Contributor

bors commented Feb 15, 2021

⌛ Testing commit 7226291 with merge 87c682a...

@bors
Copy link
Contributor

bors commented Feb 15, 2021

☀️ Test successful - checks-action_dev_test, checks-action_remark_test, checks-action_test
Approved by: Manishearth
Pushing 87c682a to master...

@bors bors merged commit 87c682a into master Feb 15, 2021
@phansch phansch deleted the update-compiletest branch February 15, 2021 19:38
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