Skip to content

Conversation

@RalfJung
Copy link
Member

@oli-obk I am not entirely sure whether this is the intended port, the types look a bit gnarly and the changelog doesn't say anything about this breakage... but oli-obk/ui_test#316 does something similar.

Please write a short comment explaining your change (or "none" for internal only changes)

changelog: none

@rustbot
Copy link
Collaborator

rustbot commented Jun 30, 2025

r? @samueltardieu

rustbot has assigned @samueltardieu.
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 Jun 30, 2025
Copy link
Member

@samueltardieu samueltardieu left a comment

Choose a reason for hiding this comment

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

LGTM, except one nit

@samueltardieu samueltardieu added this pull request to the merge queue Jul 1, 2025
Merged via the queue into rust-lang:master with commit 4c99809 Jul 1, 2025
11 checks passed
@RalfJung RalfJung deleted the ui_test branch July 1, 2025 05:39
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.

3 participants