arti: logging config: Replace a manual Debug impl

Cargo.lock

@@ -89,6 +89,7 @@ dependencies = [
  "clap",
  "config",
  "derive_builder_fork_arti",
+ "educe",
  "fs-mistrust",
  "futures",
  "itertools",

crates/arti/Cargo.toml

@@ -30,6 +30,7 @@ cfg-if = "1.0.0"
 clap = "2.33.0"
 config = { version = "0.13", default-features = false, features = ["toml"] }
 derive_builder = { version = "0.11", package = "derive_builder_fork_arti" }
+educe = "0.4.6"
 fs-mistrust = { path = "../fs-mistrust", version = "0.2.0" }
 futures = "0.3.14"
 notify = "4.0"

crates/arti/src/logging.rs

@@ -2,6 +2,7 @@
 
 use anyhow::{anyhow, Context, Result};
 use derive_builder::Builder;
+use educe::Educe;
 use fs_mistrust::Mistrust;
 use serde::{Deserialize, Serialize};
 use std::path::Path;
@@ -96,7 +97,8 @@ pub struct LogfileConfig {
 }
 
 /// How often to rotate a log file
-#[derive(Debug, Clone, Serialize, Deserialize, Copy, Eq, PartialEq)]
+#[derive(Debug, Clone, Educe, Serialize, Deserialize, Copy, Eq, PartialEq)]
+#[educe(Default)]
 #[non_exhaustive]
 #[serde(rename_all = "lowercase")]
 pub enum LogRotation {
@@ -105,15 +107,10 @@ pub enum LogRotation {
     /// Rotate logs hourly
     Hourly,
     /// Never rotate the log
+    #[educe(Default)]
     Never,
 }
 
-impl Default for LogRotation {
-    fn default() -> Self {
-        Self::Never
-    }
-}
-
 impl LogfileConfig {
     /// Return a new [`LogfileConfigBuilder`]
     pub fn builder() -> LogfileConfigBuilder {