chore(module_name_repeat): Rename stutter lint to module_name_repeat to avoid ableist language#3554
Conversation
…to avoid ableist language
|
I did not touch CHANGELOG.md so far, it looks like the links there are auto-generated? |
|
Thanks for your contribution and welcome to Clippy! 🎉 The changelog will be adapted automatically if you run The lint also needs to be registered as renamed. See rustc-docs Since until now we never registered a lint as renamed, I'm not sure where to do this exactly. Could you try to call the |
|
Thanks! Ran Tried to add the register_renamed() call at the end after all lints are registered, but that fails with "invalid lint renaming of stutter to module_name_repeat". What is the correct name of the new module_name_repeat lint so that it is valid here? Maybe it is not in the store where I'm running the register_renamed() on? How can I get the store which has the new module_name_repeat lint? |
|
Yeah I thought of recalling something, that the lints actually get added later in the compiler. I need to give this a closer look tomorrow. :) |
|
We could also deprecate the old one with a deprecation message that mentions the new lint name, if |
clippy_lints/src/lib.rs
Outdated
| unwrap::UNNECESSARY_UNWRAP, | ||
| ]); | ||
|
|
||
| store.register_renamed("stutter", "module_name_repeat"); |
There was a problem hiding this comment.
maybe we need clippy::stutter here?
|
Just a bikeshed now: I think we need to name it |
|
r=me with everyone involved alright with the new name. |
This comment has been minimized.
This comment has been minimized.
chore(module_name_repeat): Rename stutter lint to module_name_repeat to avoid ableist language See #3521
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
|
📌 Commit d74288e has been approved by |
chore(module_name_repeat): Rename stutter lint to module_name_repeat to avoid ableist language See #3521
|
☀️ Test successful - status-appveyor, status-travis |
See #3521