-
-
Notifications
You must be signed in to change notification settings - Fork 14.1k
Tidying up tests/ui/issues tests [1/N]
#149498
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
|
This PR modifies |
|
r? @fee1-dead rustbot has assigned @fee1-dead. Use |
This comment has been minimized.
This comment has been minimized.
|
Apologies for the late review. r=me after squash. @bors rollup delegate+ |
|
✌️ @reddevilmidzy, you can now approve this pull request! If @fee1-dead told you to " |
merged tests/ui/issues/issue-2951.rs with tests/ui/type/type-parameter-names.rs Merged tests/ui/for-loop-while/break-continue-in-loop-while-contiditoin-1.rs with tests/ui/for-loop-while/break-continue-in-loop-while-contiditoin-2.rs Removed tests/ui/issues/issue-2383.rs duplicated of library\alloc\src\collections\vec_deque\tests.rs Removed tests/ui/issues/issue-20714.rs duplicated of tests/ui/empty/empty-struct-unit-expr.rs Added comment to tests/ui/match/match-option-result-mismatch.rs, tests/ui/numeric/ref-int.rs, tests/ui/box/self-assignment.rs
|
No problem, thanks for the review :) |
Rollup of 4 pull requests Successful merges: - #147136 (Add warn-by-default lint for visibility on `const _` declarations) - #149498 (Tidying up `tests/ui/issues` tests [1/N]) - #149721 (rustc book: fix `*-pc-windows-msvc` link label in sidebar) - #149724 (Fix off-by-one staging output when testing the library) r? `@ghost` `@rustbot` modify labels: rollup
Note
Intermediate commits are intended to help review, but will be squashed add comment commit prior to merge.
part of #133895