-
-
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 |
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