Skip to content

Conversation

oli-obk
Copy link
Contributor

@oli-obk oli-obk commented Jul 15, 2024

r? @Alexendoo

the rustfix diff is caused by oli-obk/ui_test#244

This should solve the issues around missing summaries at the end

changelog: none

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label Jul 15, 2024
@oli-obk
Copy link
Contributor Author

oli-obk commented Jul 15, 2024

cc @GuillaumeGomez #11421

@GuillaumeGomez
Copy link
Member

Awesome, thank you!

@flip1995
Copy link
Member

I opened #13099 to fix the @no-rustfix attributes that were added. The same thing already worked great with #12379

@Alexendoo
Copy link
Member

🎉

@bors r+

@bors
Copy link
Contributor

bors commented Jul 15, 2024

📌 Commit 3b390d4 has been approved by Alexendoo

It is now in the queue for this repository.

@bors
Copy link
Contributor

bors commented Jul 15, 2024

⌛ Testing commit 3b390d4 with merge 22eeb11...

@bors
Copy link
Contributor

bors commented Jul 15, 2024

☀️ Test successful - checks-action_dev_test, checks-action_remark_test, checks-action_test
Approved by: Alexendoo
Pushing 22eeb11 to master...

1 similar comment
@bors
Copy link
Contributor

bors commented Jul 15, 2024

☀️ Test successful - checks-action_dev_test, checks-action_remark_test, checks-action_test
Approved by: Alexendoo
Pushing 22eeb11 to master...

@bors bors merged commit 22eeb11 into rust-lang:master Jul 15, 2024
8 checks passed
@bors
Copy link
Contributor

bors commented Jul 15, 2024

👀 Test was successful, but fast-forwarding failed: 422 Changes must be made through a pull request.

@oli-obk oli-obk deleted the bump_ui_test branch July 15, 2024 14:39
@GuillaumeGomez
Copy link
Member

Sorry but the bug is still there:

thread 'main' panicked at /home/imperio/.cargo/registry/src/index.crates.io-6f17d22bba15001f/annotate-snippets-0.11.4/src/renderer/display_list.rs:1103:29:
byte index 18 is not a char boundary; it is inside '脑' (bytes 16..19) of `    let _ = "电脑\0".as_ptr();`
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

bors added a commit that referenced this pull request Aug 12, 2024
Add a test for ice-3717.rs

this PR is a part of #13099.

Based on the changes introduced in #13098 for introduce ui_test, we will update the uitest output.
This is a fix for `ice-3717.rs`.

Although fixes have already been made in #13216, it seems that he is a first-time contributor.
I thought it might be better for him to refer to my PR, so I created it accordingly.

Since this is my first contribution in a while, please let me know if there are any issues or required changes.

changelog:
None

r! `@flip1995`
bors added a commit that referenced this pull request Aug 12, 2024
Add a test for ice-3717.rs

this PR is a part of #13099.

Based on the changes introduced in #13098 for introduce ui_test, we will update the uitest output.
This is a fix for `ice-3717.rs`.

Although fixes have already been made in #13216, it seems that he is a first-time contributor.
I thought it might be better for him to refer to my PR, so I created it accordingly.

Since this is my first contribution in a while, please let me know if there are any issues or required changes.

changelog: none

r! `@flip1995`
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