Skip to content

Conversation

@GuillaumeGomez
Copy link
Member

Fixes #151006 (hopefully).

Instead of asserting right away, it waits for the element count to be the one expected.

r? @jieyouxu

@rustbot rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-rustdoc Relevant to the rustdoc team, which will review and decide on the PR/issue. T-rustdoc-frontend Relevant to the rustdoc-frontend team, which will review and decide on the web UI/UX output. labels Jan 13, 2026
Copy link
Member

@jieyouxu jieyouxu left a comment

Choose a reason for hiding this comment

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

Thank you, hopefully this helps

🙏

View changes since this review

@jieyouxu
Copy link
Member

@bors r+ rollup

@rust-bors rust-bors bot added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Jan 13, 2026
@rust-bors
Copy link
Contributor

rust-bors bot commented Jan 13, 2026

📌 Commit 62849e6 has been approved by jieyouxu

It is now in the queue for this repository.

JonathanBrouwer added a commit to JonathanBrouwer/rust that referenced this pull request Jan 13, 2026
Reduce flakyness for `tests/rustdoc-gui/notable-trait.goml`

Fixes rust-lang#151006 (hopefully).

Instead of asserting right away, it waits for the element count to be the one expected.

r? @jieyouxu
JonathanBrouwer added a commit to JonathanBrouwer/rust that referenced this pull request Jan 13, 2026
Reduce flakyness for `tests/rustdoc-gui/notable-trait.goml`

Fixes rust-lang#151006 (hopefully).

Instead of asserting right away, it waits for the element count to be the one expected.

r? @jieyouxu
rust-bors bot pushed a commit that referenced this pull request Jan 13, 2026
…uwer

Rollup of 13 pull requests

Successful merges:

 - #150586 (rustdoc: Fix intra-doc link bugs involving type aliases and associated items)
 - #150677 (Improve std::path::Path::join documentation)
 - #150723 (std: move `errno` and related functions into `sys::io`)
 - #150737 (diagnostics: make implicit Sized bounds explicit in E0277)
 - #150771 (Remove legacy homu `try` and `auto` branch mentions)
 - #150915 (Regression test for type params on eii)
 - #150962 (Remove `FeedConstTy` and provide ty when lowering const arg)
 - #151017 (Port the rustc dump attributes to the attribute parser)
 - #151019 (Make `Type::of` support unsized types)
 - #151034 (std: Change UEFI env vars to volatile storage)
 - #151052 (ui: add regression test for macro resolution ICE (issue #150711))
 - #151053 (Reduce flakyness for `tests/rustdoc-gui/notable-trait.goml`)
 - #151055 (Emit error instead of delayed bug when meeting mismatch type for const array)

r? @ghost
GuillaumeGomez added a commit to GuillaumeGomez/rust that referenced this pull request Jan 13, 2026
Reduce flakyness for `tests/rustdoc-gui/notable-trait.goml`

Fixes rust-lang#151006 (hopefully).

Instead of asserting right away, it waits for the element count to be the one expected.

r? @jieyouxu
rust-bors bot pushed a commit that referenced this pull request Jan 14, 2026
Rollup of 13 pull requests

Successful merges:

 - #150587 (triagebot: add A-rustdoc-js autolabel)
 - #150677 (Improve std::path::Path::join documentation)
 - #150737 (diagnostics: make implicit Sized bounds explicit in E0277)
 - #150771 (Remove legacy homu `try` and `auto` branch mentions)
 - #150840 (Make `--print=check-cfg` output compatible `--check-cfg` arguments)
 - #150915 (Regression test for type params on eii)
 - #151017 (Port the rustc dump attributes to the attribute parser)
 - #151019 (Make `Type::of` support unsized types)
 - #151031 (Support arrays in type reflection)
 - #151043 (armv7-unknown-linux-uclibceabihf.md: Fix bootstrap.toml syntax)
 - #151052 (ui: add regression test for macro resolution ICE (issue #150711))
 - #151053 (Reduce flakyness for `tests/rustdoc-gui/notable-trait.goml`)
 - #151055 (Emit error instead of delayed bug when meeting mismatch type for const array)

r? @ghost
@rust-bors rust-bors bot merged commit 860e60a into rust-lang:main Jan 14, 2026
11 checks passed
@rustbot rustbot added this to the 1.94.0 milestone Jan 14, 2026
rust-timer added a commit that referenced this pull request Jan 14, 2026
Rollup merge of #151053 - reduce-gui-test-flakyness, r=jieyouxu

Reduce flakyness for `tests/rustdoc-gui/notable-trait.goml`

Fixes #151006 (hopefully).

Instead of asserting right away, it waits for the element count to be the one expected.

r? @jieyouxu
@GuillaumeGomez GuillaumeGomez deleted the reduce-gui-test-flakyness branch January 14, 2026 09:08
rust-bors bot pushed a commit that referenced this pull request Jan 15, 2026
…eyouxu

Reduce rustdoc GUI flakyness, take 2

Fixes #151006 (hopefully).

Seems like #151053 did not fix the flakyness issue, so here is another take on this issue.

r? @jieyouxu
rust-bors bot pushed a commit that referenced this pull request Jan 15, 2026
…eyouxu

Reduce rustdoc GUI flakyness, take 2

Fixes #151006 (hopefully).

Seems like #151053 did not fix the flakyness issue, so here is another take on this issue.

r? @jieyouxu
Zalathar added a commit to Zalathar/rust that referenced this pull request Jan 15, 2026
Reduce rustdoc GUI flakyness, take 2

Fixes rust-lang#151006 (hopefully).

Seems like rust-lang#151053 did not fix the flakyness issue, so here is another take on this issue.

r? @jieyouxu
rust-timer added a commit that referenced this pull request Jan 15, 2026
Rollup merge of #151145 - reduce-rustdoc-flakyness, r=jieyouxu

Reduce rustdoc GUI flakyness, take 2

Fixes #151006 (hopefully).

Seems like #151053 did not fix the flakyness issue, so here is another take on this issue.

r? @jieyouxu
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. T-rustdoc Relevant to the rustdoc team, which will review and decide on the PR/issue. T-rustdoc-frontend Relevant to the rustdoc-frontend team, which will review and decide on the web UI/UX output.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

tests/rustdoc-gui/notable-trait.goml is sometimes flaky

3 participants