| Age | Commit message (Collapse) | Author | Lines |
|
|
|
|
|
|
|
|
|
Co-Authored-By: Philipp Krones <hello@philkrones.com>
|
|
|
|
This adds a check to `clippy_dev` that enforces a maximum line count for
`stderr` files. CI will fail if the line count is exceeded. It's
currently set to `320` lines.
Ideally this would be implemented in `compiletest-rs` but there are
plans to move Rust's `compiletest` into the `compiletest-rs` repository
and I don't want to do the work in `compiletest` twice. However, I also
don't want to wait until the move is done, so I added the check to
`clippy_dev` until it makes sense to add it to compiletest-rs.
cc #2038
|
|
+ Print help if no subcommand is supplied
+ Make a short version of `update_lints` help for the subcommand listing
|
|
Discussion previously happened in https://github.com/rust-lang/rust/pull/43498
|
|
|
|
|
|
|
|
Co-Authored-By: phansch <dev@phansch.net>
|
|
This finishes up the rewrite of `update_lints.py` in Rust. More
specifically, this
* adds the `--check` flag and handling to clippy_dev
* tracks file changes over the different calls to `replace_region_in_file`
* only writes changes to files if the `--check` flag is *not* used
* runs `./util/dev update_lints --check` on CI instead of the old script
* replaces usage of the `update_lints.py` script with an error
`./util/dev update_lints` behaves 99% the same as the python script.
The only difference that I'm aware of is an ordering change to
`clippy_lints/src/lib.rs` because underscores seem to be sorted
differently in Rust and in Python.
:checkered_flag:
|
|
|
|
|
|
|
|
|
|
|
|
The update script now also generates the 'register_removed' section in
`clippy_lints/src/lib.rs`.
Also, instead of using `let mut store ...`, I added a new identifier
line so that the replacement will continue to work in case `let mut
store ...` ever changes.
|
|
This now also updates the link list at the bottom of the changelog.
|
|
This allows the usage of `util/dev update_lints` which will write the
new lint_count to the `README.md`.
|
|
Because now `usable_lints` will also exclude internal lints.
|
|
|
|
Small refactoring pulled out of work on #3266.
This should make the methods a bit more flexible.
|
|
|
|
This makes the API of `lib.rs` a bit more flexible.
|
|
|
|
|