-
-
Notifications
You must be signed in to change notification settings - Fork 14.5k
Remove the 4 failing tests from rustdoc-gui #152194
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
Conversation
|
Some changes occurred in GUI tests. |
|
Tracked in #152197. Given that a contributor said they can reproduce test failures of these 4 tests locally (#t-infra > CI failure because `tests/rustdoc-gui/search-filter.goml` @ 💬), I am going to r+ this to try to unblock the tree. Hopefully it at least reduces the likelihood down from "every PR is hitting these" to "occassionally". @bors r+ |
Rollup of 4 pull requests Successful merges: - #152174 (stdarch subtree update) - #151278 (Provide more context on trait bounds being unmet due to imperfect derive) - #151955 (escape symbol names in global asm) - #152194 (Remove the 4 failing tests from rustdoc-gui) Failed merges: - #152191 (Convert to inline diagnostics in `rustc_hir_analysis`)
Rollup merge of #152194 - JonathanBrouwer:fix-rustdoc-gui, r=jieyouxu Remove the 4 failing tests from rustdoc-gui These are the 4 tests that @Bryntet got to fail locally See [#t-infra > CI failure because `tests/rustdoc-gui/search-filter.goml`](https://rust-lang.zulipchat.com/#narrow/channel/242791-t-infra/topic/CI.20failure.20because.20.60tests.2Frustdoc-gui.2Fsearch-filter.2Egoml.60/with/572271674) r? @ghost
|
Isn't there a way to disable rather than remove them? Makes it easier to revert this later. |
I don't think this is implemented; |
|
We need to unblock beta too, so: @rustbot label beta-accepted |
|
Marked it as accepted for stable backport as well. |
[stable] prepare Rust 1.93.1 This includes the following backported fixes: - Don't try to recover keyword as non-keyword identifier #150590 - Fix `panicking_unwrap` FP on field access with implicit deref rust-lang/rust-clippy#16196 - Revert "Update wasm-related dependencies in CI" #152259 And these are just to avoid recent CI issues: - Remove rustdoc GUI flaky test #152116 - Remove the 4 failing tests from rustdoc-gui #152194 r? @rust-lang/release @rustbot ping relnotes-interest-group
[stable] prepare Rust 1.93.1 This includes the following backported fixes: - Don't try to recover keyword as non-keyword identifier #150590 - Fix `panicking_unwrap` FP on field access with implicit deref rust-lang/rust-clippy#16196 - Revert "Update wasm-related dependencies in CI" #152259 And these are just to avoid recent CI issues: - Remove rustdoc GUI flaky test #152116 - Remove the 4 failing tests from rustdoc-gui #152194 r? @rust-lang/release @rustbot ping relnotes-interest-group
These are the 4 tests that @Bryntet got to fail locally
See #t-infra > CI failure because `tests/rustdoc-gui/search-filter.goml`
r? @ghost