Skip to content
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

UI Tests: Separate suspicious_else_formatting tests #6473

Merged
merged 1 commit into from
Dec 19, 2020

Conversation

phansch
Copy link
Member

@phansch phansch commented Dec 19, 2020

Was briefly looking into #3864 when I saw that the tests could benefit from being in their own file.


changelog: none

@rust-highfive
Copy link

r? @flip1995

(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 Dec 19, 2020
@flip1995
Copy link
Member

@bors r+

Thanks!

@bors
Copy link
Collaborator

bors commented Dec 19, 2020

📌 Commit 8ddf4ce has been approved by flip1995

@bors
Copy link
Collaborator

bors commented Dec 19, 2020

⌛ Testing commit 8ddf4ce with merge 0904f54...

@bors
Copy link
Collaborator

bors commented Dec 19, 2020

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

@bors bors merged commit 0904f54 into rust-lang:master Dec 19, 2020
@phansch phansch deleted the split-up-ui-test branch December 26, 2020 14:29
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.

4 participants