rename lint

Author: SpecificProtagonist

clippy_lints/src/error_impl_error.rs

@@ -10,7 +10,7 @@ use rustc_span::sym;
 
 declare_clippy_lint! {
     /// ### What it does
-    /// Checks for types named `Error` that implement `Error`.
+    /// Checks for types named `*Error` that implement `Error`.
     ///
     /// ### Why restrict this?
     /// It can become confusing when a codebase has 20 types all named `Error`, requiring either

clippy_lints/src/missing_error_impl.rs clippy_lints/src/missing_error_implementations.rs

@@ -31,14 +31,14 @@ declare_clippy_lint! {
     /// impl core::error::Error for ParseError { ... }
     /// ```
     #[clippy::version = "1.87.0"]
-    pub MISSING_ERROR_IMPL,
+    pub MISSING_ERROR_IMPLEMENTATIONS,
     suspicious,
     "exported types with potentially forgotten `Error` implementation"
 }
 
-declare_lint_pass!(MissingErrorImpl => [MISSING_ERROR_IMPL]);
+declare_lint_pass!(MissingErrorImplementations => [MISSING_ERROR_IMPLEMENTATIONS]);
 
-impl<'tcx> LateLintPass<'tcx> for MissingErrorImpl {
+impl<'tcx> LateLintPass<'tcx> for MissingErrorImplementations {
     fn check_item(&mut self, cx: &LateContext<'tcx>, item: &'tcx Item<'tcx>) {
         match item.kind {
             ItemKind::Enum(_, generics) | ItemKind::Struct(_, generics) => {
@@ -59,7 +59,7 @@ impl<'tcx> LateLintPass<'tcx> for MissingErrorImpl {
                 {
                     span_lint(
                         cx,
-                        MISSING_ERROR_IMPL,
+                        MISSING_ERROR_IMPLEMENTATIONS,
                         item.ident.span,
                         "error type doesn't implement `Error`",
                     );

tests/ui/missing_error_impl.rs tests/ui/missing_error_implementations.rs

@@ -1,11 +1,11 @@
-#![warn(clippy::missing_error_impl)]
+#![warn(clippy::missing_error_implementations)]
 
 struct InternalError;
 
 pub(crate) struct CrateInternalError;
 
 pub struct PublicError;
-//~^ missing_error_impl
+//~^ missing_error_implementations
 
 pub struct NotAnErrorType;
 

tests/ui/missing_error_impl.stderr tests/ui/missing_error_implementations.stderr

@@ -1,11 +1,11 @@
 error: error type doesn't implement `Error`
-  --> tests/ui/missing_error_impl.rs:7:12
+  --> tests/ui/missing_error_implementations.rs:7:12
    |
 LL | pub struct PublicError;
    |            ^^^^^^^^^^^
    |
-   = note: `-D clippy::missing-error-impl` implied by `-D warnings`
-   = help: to override `-D warnings` add `#[allow(clippy::missing_error_impl)]`
+   = note: `-D clippy::missing-error-implementations` implied by `-D warnings`
+   = help: to override `-D warnings` add `#[allow(clippy::missing_error_implementations)]`
 
 error: aborting due to 1 previous error