chore: bump derive_more to v1.0

Cargo.toml

@@ -31,7 +31,7 @@ serde = { version = "1.0.195", features = ["derive"] }
 serde_json = "1.0.111"
 regex = "1.10.2"
 once_cell = "1.19.0"
-derive_more = { version = "0.99.17", features = ["display"] }
+derive_more = { version = "1.0", features = ["display"] }
 anyhow = "1.0.79"
 if_chain = "1.0.2"
 phf = { version = "0.11.2", features = ["macros"] }

src/rules/adjacent_overload_signatures.rs

@@ -15,13 +15,13 @@ const CODE: &str = "adjacent-overload-signatures";
 
 #[derive(Display)]
 enum AdjacentOverloadSignaturesMessage {
-  #[display(fmt = "All `{}` signatures should be adjacent", _0)]
+  #[display("All `{}` signatures should be adjacent", _0)]
   ShouldBeAdjacent(String),
 }
 
 #[derive(Display)]
 enum AdjacentOverloadSignaturesHint {
-  #[display(fmt = "Make sure all overloaded signatures are grouped together")]
+  #[display("Make sure all overloaded signatures are grouped together")]
   GroupedTogether,
 }
 
@@ -204,13 +204,13 @@ impl<'a> ExtractMethod for ast_view::TsTypeElement<'a> {
 #[derive(PartialEq, Eq, Hash, Clone, Display)]
 #[allow(clippy::enum_variant_names)]
 enum Method {
-  #[display(fmt = "{}", _0)]
+  #[display("{}", _0)]
   Method(String),
-  #[display(fmt = "{}", _0)]
+  #[display("{}", _0)]
   Static(String),
-  #[display(fmt = "call")]
+  #[display("call")]
   CallSignature,
-  #[display(fmt = "new")]
+  #[display("new")]
   ConstructSignature,
 }
 

src/rules/default_param_last.rs

@@ -13,15 +13,13 @@ const CODE: &str = "default-param-last";
 
 #[derive(Display)]
 enum DefaultParamLastMessage {
-  #[display(fmt = "default parameters should be at last")]
+  #[display("default parameters should be at last")]
   DefaultLast,
 }
 
 #[derive(Display)]
 enum DefaultParamLastHint {
-  #[display(
-    fmt = "Modify the signatures to move default parameter(s) to the end"
-  )]
+  #[display("Modify the signatures to move default parameter(s) to the end")]
   MoveToEnd,
 }
 

src/rules/eqeqeq.rs

@@ -14,17 +14,17 @@ const CODE: &str = "eqeqeq";
 
 #[derive(Display)]
 enum EqeqeqMessage {
-  #[display(fmt = "expected '===' and instead saw '=='.")]
+  #[display("expected '===' and instead saw '=='.")]
   ExpectedEqual,
-  #[display(fmt = "expected '!==' and instead saw '!='.")]
+  #[display("expected '!==' and instead saw '!='.")]
   ExpectedNotEqual,
 }
 
 #[derive(Display)]
 enum EqeqeqHint {
-  #[display(fmt = "Use '==='")]
+  #[display("Use '==='")]
   UseEqeqeq,
-  #[display(fmt = "Use '!=='")]
+  #[display("Use '!=='")]
   UseNoteqeq,
 }
 

src/rules/explicit_function_return_type.rs

@@ -13,13 +13,13 @@ const CODE: &str = "explicit-function-return-type";
 
 #[derive(Display)]
 enum ExplicitFunctionReturnTypeMessage {
-  #[display(fmt = "Missing return type on function")]
+  #[display("Missing return type on function")]
   MissingRetType,
 }
 
 #[derive(Display)]
 enum ExplicitFunctionReturnTypeHint {
-  #[display(fmt = "Add a return type to the function signature")]
+  #[display("Add a return type to the function signature")]
   AddRetType,
 }
 

src/rules/explicit_module_boundary_types.rs

@@ -13,19 +13,19 @@ const CODE: &str = "explicit-module-boundary-types";
 
 #[derive(Display)]
 enum ExplicitModuleBoundaryTypesMessage {
-  #[display(fmt = "Missing return type on function")]
+  #[display("Missing return type on function")]
   MissingRetType,
 
-  #[display(fmt = "All arguments should be typed")]
+  #[display("All arguments should be typed")]
   MissingArgType,
 }
 
 #[derive(Display)]
 enum ExplicitModuleBoundaryTypesHint {
-  #[display(fmt = "Add a return type to the function signature")]
+  #[display("Add a return type to the function signature")]
   AddRetType,
 
-  #[display(fmt = "Add types to all the function arguments")]
+  #[display("Add types to all the function arguments")]
   AddArgTypes,
 }
 

src/rules/getter_return.rs

@@ -26,15 +26,15 @@ const CODE: &str = "getter-return";
 
 #[derive(Display)]
 enum GetterReturnMessage {
-  #[display(fmt = "Expected to return a value in '{}'.", _0)]
+  #[display("Expected to return a value in '{}'.", _0)]
   Expected(String),
-  #[display(fmt = "Expected '{}' to always return a value.", _0)]
+  #[display("Expected '{}' to always return a value.", _0)]
   ExpectedAlways(String),
 }
 
 #[derive(Display)]
 enum GetterReturnHint {
-  #[display(fmt = "Return a value from the getter function")]
+  #[display("Return a value from the getter function")]
   Return,
 }
 

src/rules/no_compare_neg_zero.rs

@@ -19,13 +19,13 @@ const CODE: &str = "no-compare-neg-zero";
 
 #[derive(Display)]
 enum NoCompareNegZeroMessage {
-  #[display(fmt = "Do not compare against -0")]
+  #[display("Do not compare against -0")]
   Unexpected,
 }
 
 #[derive(Display)]
 enum NoCompareNegZeroHint {
-  #[display(fmt = "Use Object.is(x, -0) instead")]
+  #[display("Use Object.is(x, -0) instead")]
   ObjectIs,
 }
 

src/rules/no_cond_assign.rs

@@ -14,16 +14,14 @@ const CODE: &str = "no-cond-assign";
 
 #[derive(Display)]
 enum NoCondAssignMessage {
-  #[display(
-    fmt = "Expected a conditional expression and instead saw an assignment"
-  )]
+  #[display("Expected a conditional expression and instead saw an assignment")]
   Unexpected,
 }
 
 #[derive(Display)]
 enum NoCondAssignHint {
   #[display(
-    fmt = "Change assignment (`=`) to comparison (`===`) or move assignment out of condition"
+    "Change assignment (`=`) to comparison (`===`) or move assignment out of condition"
   )]
   ChangeOrMove,
 }

src/rules/no_const_assign.rs

@@ -17,14 +17,14 @@ const CODE: &str = "no-const-assign";
 
 #[derive(Display)]
 enum NoConstantAssignMessage {
-  #[display(fmt = "Reassigning constant variable is not allowed")]
+  #[display("Reassigning constant variable is not allowed")]
   Unexpected,
 }
 
 #[derive(Display)]
 enum NoConstantAssignHint {
   #[display(
-    fmt = "Change `const` declaration to `let` or double check the correct variable is used"
+    "Change `const` declaration to `let` or double check the correct variable is used"
   )]
   Remove,
 }

src/rules/no_constant_condition.rs

@@ -17,15 +17,13 @@ const CODE: &str = "no-constant-condition";
 
 #[derive(Display)]
 enum NoConstantConditionMessage {
-  #[display(
-    fmt = "Use of a constant expressions as conditions is not allowed."
-  )]
+  #[display("Use of a constant expressions as conditions is not allowed.")]
   Unexpected,
 }
 
 #[derive(Display)]
 enum NoConstantConditionHint {
-  #[display(fmt = "Remove the constant expression")]
+  #[display("Remove the constant expression")]
   Remove,
 }
 

src/rules/no_control_regex.rs

@@ -18,7 +18,7 @@ const CODE: &str = "no-control-regex";
 #[derive(Display)]
 enum NoControlRegexMessage {
   #[display(
-    fmt = "Unexpected control character(s) in regular expression: \\x{:x}.",
+    "Unexpected control character(s) in regular expression: \\x{:x}.",
     _0
   )]
   Unexpected(u64),
@@ -27,7 +27,7 @@ enum NoControlRegexMessage {
 #[derive(Display)]
 enum NoControlRegexHint {
   #[display(
-    fmt = "Disable the rule if the control character (\\x... or \\u00..) was intentional, otherwise rework your RegExp"
+    "Disable the rule if the control character (\\x... or \\u00..) was intentional, otherwise rework your RegExp"
   )]
   DisableOrRework,
 }

src/rules/no_debugger.rs

@@ -14,13 +14,13 @@ const CODE: &str = "no-debugger";
 
 #[derive(Display)]
 enum NoDebuggerMessage {
-  #[display(fmt = "`debugger` statement is not allowed")]
+  #[display("`debugger` statement is not allowed")]
   Unexpected,
 }
 
 #[derive(Display)]
 enum NoDebuggerHint {
-  #[display(fmt = "Remove the `debugger` statement")]
+  #[display("Remove the `debugger` statement")]
   Remove,
 }
 

src/rules/no_delete_var.rs

@@ -14,13 +14,13 @@ const CODE: &str = "no-delete-var";
 
 #[derive(Display)]
 enum NoDeleteVarMessage {
-  #[display(fmt = "Variables shouldn't be deleted")]
+  #[display("Variables shouldn't be deleted")]
   Unexpected,
 }
 
 #[derive(Display)]
 enum NoDeleteVarHint {
-  #[display(fmt = "Remove the deletion statement")]
+  #[display("Remove the deletion statement")]
   Remove,
 }
 

src/rules/no_dupe_args.rs

@@ -15,13 +15,13 @@ const CODE: &str = "no-dupe-args";
 
 #[derive(Display)]
 enum NoDupeArgsMessage {
-  #[display(fmt = "Duplicate arguments not allowed")]
+  #[display("Duplicate arguments not allowed")]
   Unexpected,
 }
 
 #[derive(Display)]
 enum NoDupeArgsHint {
-  #[display(fmt = "Rename or remove the duplicate (e.g. same name) argument")]
+  #[display("Rename or remove the duplicate (e.g. same name) argument")]
   RenameOrRemove,
 }
 

src/rules/no_dupe_class_members.rs

@@ -22,13 +22,13 @@ const CODE: &str = "no-dupe-class-members";
 
 #[derive(Display)]
 enum NoDupeClassMembersMessage {
-  #[display(fmt = "Duplicate name '{}'", _0)]
+  #[display("Duplicate name '{}'", _0)]
   Duplicate(String),
 }
 
 #[derive(Display)]
 enum NoDupeClassMembersHint {
-  #[display(fmt = "Rename or remove the function with the duplicated name")]
+  #[display("Rename or remove the function with the duplicated name")]
   RenameOrRemove,
 }
 

src/rules/no_dupe_else_if.rs

@@ -19,16 +19,14 @@ const CODE: &str = "no-dupe-else-if";
 #[derive(Display)]
 enum NoDupeElseIfMessage {
   #[display(
-    fmt = "This branch can never execute. Its condition is a duplicate or covered by previous conditions in the if-else-if chain."
+    "This branch can never execute. Its condition is a duplicate or covered by previous conditions in the if-else-if chain."
   )]
   Unexpected,
 }
 
 #[derive(Display)]
 enum NoDupeElseIfHint {
-  #[display(
-    fmt = "Remove or rework the `else if` condition which is duplicated"
-  )]
+  #[display("Remove or rework the `else if` condition which is duplicated")]
   RemoveOrRework,
 }