Auto merge of #9580 - TennyZhuang:upper_case_acronyms-enum, r=llogiq

let `upper_case_acronyms` check the enum name

Signed-off-by: TennyZhuang <zty0826@gmail.com>

Fix #9579

changelog: [`upper_case_acronyms`]: check the enum name
This commit is contained in:
bors 2022-10-03 09:01:05 +00:00
commit 18e10ca290
3 changed files with 23 additions and 1 deletions

View File

@ -114,6 +114,7 @@ fn check_item(&mut self, cx: &LateContext<'_>, it: &Item<'_>) {
check_ident(cx, &it.ident, self.upper_case_acronyms_aggressive);
},
ItemKind::Enum(ref enumdef, _) => {
check_ident(cx, &it.ident, self.upper_case_acronyms_aggressive);
// check enum variants separately because again we only want to lint on private enums and
// the fn check_variant does not know about the vis of the enum of its variants
enumdef

View File

@ -38,4 +38,13 @@ enum ParseErrorPrivate<T> {
Parse(T, String),
}
// do lint here
struct JSON;
// do lint here
enum YAML {
Num(u32),
Str(String),
}
fn main() {}

View File

@ -54,5 +54,17 @@ error: name `WASD` contains a capitalized acronym
LL | WASD(u8),
| ^^^^ help: consider making the acronym lowercase, except the initial letter: `Wasd`
error: aborting due to 9 previous errors
error: name `JSON` contains a capitalized acronym
--> $DIR/upper_case_acronyms.rs:42:8
|
LL | struct JSON;
| ^^^^ help: consider making the acronym lowercase, except the initial letter: `Json`
error: name `YAML` contains a capitalized acronym
--> $DIR/upper_case_acronyms.rs:45:6
|
LL | enum YAML {
| ^^^^ help: consider making the acronym lowercase, except the initial letter: `Yaml`
error: aborting due to 11 previous errors