Standardize on 'Error, Debug' derive order

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),

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),

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,