Standardize on 'Error, Debug' derive order

dtolnay · Dec 7, 2024 · 42b1460 · 42b1460
1 parent 2096b11
commit 42b1460
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 5 deletions.
diff --git a/tests/test_display.rs b/tests/test_display.rs
@@ -271,13 +271,13 @@ fn test_macro_rules() {
 
     macro_rules! decl_error {
         ($variant:ident($value:ident)) => {
-            #[derive(Debug, Error)]
+            #[derive(Error, Debug)]
             pub enum Error0 {
                 #[error("{0:?}")]
                 $variant($value),
             }
 
-            #[derive(Debug, Error)]
+            #[derive(Error, Debug)]
             #[error("{0:?}")]
             pub enum Error1 {
                 $variant($value),

diff --git a/tests/test_lints.rs b/tests/test_lints.rs
@@ -12,7 +12,7 @@ fn test_unused_qualifications() {
     // std::error::Error is already imported in the caller's scope so it must
     // suppress unused_qualifications.
 
-    #[derive(Debug, Error)]
+    #[derive(Error, Debug)]
     #[error("...")]
     pub struct MyError;
 
@@ -24,7 +24,7 @@ fn test_needless_lifetimes() {
     #![allow(dead_code)]
     #![deny(clippy::needless_lifetimes)]
 
-    #[derive(Debug, Error)]
+    #[derive(Error, Debug)]
     #[error("...")]
     pub enum MyError<'a> {
         A(#[from] std::io::Error),

diff --git a/tests/ui/from-not-source.rs b/tests/ui/from-not-source.rs
@@ -1,6 +1,6 @@
 use thiserror::Error;
 
-#[derive(Debug, Error)]
+#[derive(Error, Debug)]
 pub struct Error {
     #[source]
     source: std::io::Error,