Skip to content

Conversation

@oli-obk
Copy link
Contributor

@oli-obk oli-obk commented Sep 7, 2024

This should give a bunch of ui improvements when lots of tests are running. Please lmk of any issues with it. Switching to --quiet should always avoid any issues, so there's a workaround if anything crops up. Also please check that this does not regress performance for you. It doesn't for me, but that doesn't mean much.

changelog: none

@rustbot
Copy link
Collaborator

rustbot commented Sep 7, 2024

r? @Alexendoo

rustbot has assigned @Alexendoo.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label Sep 7, 2024
Comment on lines -85 to +86
/// Nested mod //~ ERROR: item has both inner and outer attributes
/// Nested mod
//~^ ERROR: item has both inner and outer attributes
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you can't have annotations inside other comments anymore.

@oli-obk oli-obk force-pushed the bump_ui_test branch 2 times, most recently from f58089f to 78a7847 Compare September 8, 2024 13:31
@alex-semenyuk
Copy link
Member

Test failing

FAILURES:
    tests/ui/almost_complete_range.fixed

@oli-obk
Copy link
Contributor Author

oli-obk commented Sep 10, 2024

Everything works now

@flip1995
Copy link
Member

Alex is currently busy I think, so jumping in.

r? @flip1995

@rustbot rustbot assigned flip1995 and unassigned Alexendoo Sep 10, 2024
@flip1995
Copy link
Member

Thanks!

@bors r+

@bors
Copy link
Contributor

bors commented Sep 10, 2024

📌 Commit 1f13633 has been approved by flip1995

It is now in the queue for this repository.

@bors
Copy link
Contributor

bors commented Sep 10, 2024

⌛ Testing commit 1f13633 with merge 38475d2...

bors added a commit that referenced this pull request Sep 10, 2024
Bump ui_test

This should give a bunch of ui improvements when lots of tests are running. Please lmk of any issues with it. Switching to `--quiet` should always avoid any issues, so there's a workaround if anything crops up. Also please check that this does not regress performance for you. It doesn't for me, but that doesn't mean much.
@bors
Copy link
Contributor

bors commented Sep 10, 2024

💔 Test failed - checks-action_test

@flip1995
Copy link
Member

@bors retry (changelog missing)

@bors
Copy link
Contributor

bors commented Sep 10, 2024

⌛ Testing commit 1f13633 with merge a53614a...

@bors
Copy link
Contributor

bors commented Sep 10, 2024

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

@bors bors merged commit a53614a into rust-lang:master Sep 10, 2024
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.

6 participants