Standardize derive order

Author: wetheredge

src/common.rs

@@ -31,7 +31,7 @@ impl Default for Permissions {
 
 /// "Base" permissions, where all individual permissions are configured
 /// with a blanket setting.
-#[derive(Deserialize, Default, Debug, PartialEq)]
+#[derive(Deserialize, Debug, Default, PartialEq)]
 #[serde(rename_all = "kebab-case")]
 pub enum BasePermission {
     /// Whatever default permissions come from the workflow's `GITHUB_TOKEN`.
@@ -44,7 +44,7 @@ pub enum BasePermission {
 }
 
 /// A singular permission setting.
-#[derive(Deserialize, Default, Debug, PartialEq)]
+#[derive(Deserialize, Debug, Default, PartialEq)]
 #[serde(rename_all = "kebab-case")]
 pub enum Permission {
     /// Read access.
@@ -68,7 +68,7 @@ pub type Env = IndexMap<String, EnvValue>;
 /// This type also gets used for other places where GitHub Actions
 /// contextually reinterprets a YAML value as a string, e.g. trigger
 /// input values.
-#[derive(Serialize, Deserialize, Debug, PartialEq)]
+#[derive(Deserialize, Serialize, Debug, PartialEq)]
 #[serde(untagged)]
 pub enum EnvValue {
     // Missing values are empty strings.
@@ -92,7 +92,7 @@ impl Display for EnvValue {
 /// key can have either a scalar value or an array of values.
 ///
 /// This only appears internally, as an intermediate type for `scalar_or_vector`.
-#[derive(Debug, Deserialize, PartialEq)]
+#[derive(Deserialize, Debug, PartialEq)]
 #[serde(untagged)]
 enum SoV<T> {
     One(T),
@@ -119,7 +119,7 @@ where
 /// A bool or string. This is useful for cases where GitHub Actions contextually
 /// reinterprets a YAML boolean as a string, e.g. `run: true` really means
 /// `run: 'true'`.
-#[derive(Debug, Deserialize, PartialEq)]
+#[derive(Deserialize, Debug, PartialEq)]
 #[serde(untagged)]
 enum BoS {
     Bool(bool),
@@ -138,7 +138,7 @@ impl From<BoS> for String {
 /// An `if:` condition in a job or action definition.
 ///
 /// These are either booleans or bare (i.e. non-curly) expressions.
-#[derive(Debug, Deserialize, PartialEq, Serialize)]
+#[derive(Deserialize, Serialize, Debug, PartialEq)]
 #[serde(untagged)]
 pub enum If {
     Bool(bool),

src/common/expr.rs

@@ -69,7 +69,7 @@ impl<'de> Deserialize<'de> for ExplicitExpr {
 /// A "literal or expr" type, for places in GitHub Actions where a
 /// key can either have a literal value (array, object, etc.) or an
 /// expression string.
-#[derive(Debug, Deserialize, PartialEq, Serialize)]
+#[derive(Deserialize, Serialize, Debug, PartialEq)]
 #[serde(untagged)]
 pub enum LoE<T> {
     // Observe that `Expr` comes first, since `LoE<String>` should always

src/workflow/event.rs

@@ -57,7 +57,7 @@ pub enum BareEvent {
 /// Workflow event triggers, with bodies.
 ///
 /// Like [`BareEvent`], but with per-event properties.
-#[derive(Default, Deserialize, Serialize, Debug)]
+#[derive(Deserialize, Serialize, Debug, Default)]
 #[serde(default, rename_all = "snake_case")]
 pub struct Events {
     pub branch_protection_rule: OptionalBody<GenericEvent>,
@@ -157,7 +157,7 @@ impl Events {
 /// between the non-presence of an event (no trigger) and the presence
 /// of an empty event body (e.g. `pull_request:`), which means "trigger
 /// with the defaults for this event type."
-#[derive(Default, Serialize, Debug)]
+#[derive(Serialize, Debug, Default)]
 pub enum OptionalBody<T> {
     Default,
     #[default]

src/workflow/job.rs

@@ -38,7 +38,7 @@ pub struct NormalJob {
     pub services: IndexMap<String, Container>,
 }
 
-#[derive(Debug, Deserialize, PartialEq)]
+#[derive(Deserialize, Debug, PartialEq)]
 #[serde(rename_all = "kebab-case", untagged, remote = "Self")]
 pub enum RunsOn {
     #[serde(deserialize_with = "crate::common::scalar_or_vector")]