diff options
| author | Matthias Krüger <matthias.krueger@famsik.de> | 2024-04-10 04:27:39 +0200 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2024-04-10 04:27:39 +0200 |
| commit | c14b468cca583f194bae0329f88397b0974a2a69 (patch) | |
| tree | d3e9266c85380fb3966190d3464ce8d85dfbe0ee /compiler/rustc_llvm/llvm-wrapper/RustWrapper.cpp | |
| parent | 7dd24d88abcd85df56c149f66b5a8a45847947a4 (diff) | |
| parent | cbf150177f367e09156ee0e17d989b990f98c2f8 (diff) | |
| download | rust-c14b468cca583f194bae0329f88397b0974a2a69.tar.gz rust-c14b468cca583f194bae0329f88397b0974a2a69.zip | |
Rollup merge of #123568 - Oneirical:delete-tests, r=wesleywiser
Clean up tests/ui by removing `does-nothing.rs` In [a previous PR](https://github.com/rust-lang/rust/pull/123297#issuecomment-2039887806), it was suggested that this test be removed: > it's testing a basic diagnostic for an unknown variable (added over a decade ago for https://github.com/rust-lang/rust/issues/154) that is already covered by probably dozens or hundreds of other tests. It was then suggested that [opening a new PR](https://github.com/rust-lang/rust/pull/123563#discussion_r1554654102) for this would be more organized. I'm setting this as a draft, as: 1. The tests/ui directory is rather disorganized, a large quantity of tests are not even contained inside their own directories. This PR could turn into "clean up the UI tests directory", if I were to place everything into categories (for example, everything related to CLI flags could get placed in a cli directory). 2. This will have a merge conflict with #123563 should that get merged. I trust that _this time_, I won't run into [The Incident](https://github.com/rust-lang/rust/pull/123297#issuecomment-2041137569) while rebasing. Edit: Yay, I did it properly!
Diffstat (limited to 'compiler/rustc_llvm/llvm-wrapper/RustWrapper.cpp')
0 files changed, 0 insertions, 0 deletions