Tune lints for 1.87 Rust

Author: tyranron

.clippy.toml

@@ -8,6 +8,8 @@ absolute-paths-allowed-crates = [
     "prometheus",
 ]
 
+allow-expect-in-tests = true
+
 standard-macro-braces = [
     { name = "assert", brace = "(" },
     { name = "assert_eq", brace = "(" },

CONTRIBUTING.md

@@ -25,7 +25,7 @@ Additional rules, not handled by [rustfmt] and [Clippy] are described below.
 
 ```rust
 #[allow(clippy::mut_mut)]
-#[derive(smart_default::SmartDefault, Debug, Deserialize, Serialize)]
+#[derive(Debug, Deserialize, Serialize, smart_default::SmartDefault)]
 #[serde(deny_unknown_fields)]
 struct User {
     #[serde(default)]
@@ -37,7 +37,7 @@ struct User {
 
 ```rust
 #[serde(deny_unknown_fields)]
-#[derive(smart_default::SmartDefault, Debug, Deserialize, Serialize)]
+#[derive(Debug, Deserialize, Serialize, smart_default::SmartDefault)]
 #[allow(clippy::mut_mut)]
 struct User {
     id: u64,
@@ -51,6 +51,13 @@ struct User {
 }
 ```
 
+```rust
+#[derive(smart_default::SmartDefault, Debug, Deserialize, Serialize)]
+struct User {
+    id: u64,
+}
+```
+
 
 ### Markdown in docs
 
@@ -64,9 +71,9 @@ Other **code definitions** should be **referred via ```[`Entity`]``` marking** (
 
 ```rust
 /// Type of [`User`]'s unique identifier.
-/// 
+///
 /// # Constraints
-/// 
+///
 /// - It **must not be zero**.
 /// - It _should not_ overflow [`i64::MAX`] due to usage in database.
 struct UserId(u64);
@@ -78,9 +85,9 @@ struct UserId(u64);
 
     ```rust
     /// Type of [`User`]'s unique identifier.
-    /// 
+    ///
     /// ## Constraints
-    /// 
+    ///
     /// - It **must not be zero**.
     /// - It _should not_ overflow [`i64::MAX`] due to usage in database.
     struct UserId(u64);
@@ -90,9 +97,9 @@ struct UserId(u64);
 
     ```rust
     /// Type of User's unique identifier.
-    /// 
+    ///
     /// # Constraints
-    /// 
+    ///
     /// - It **must not be zero**.
     /// - It _should not_ overflow `i64::MAX` due to usage in database.
     struct UserId(u64);
@@ -102,9 +109,9 @@ struct UserId(u64);
 
     ```rust
     /// Type of [`User`]'s unique identifier.
-    /// 
+    ///
     /// # Constraints
-    /// 
+    ///
     /// - It __must not be zero__.
     /// - It *should not* overflow [`i64::MAX`] due to usage in database.
     struct UserId(u64);

src/lib.rs

@@ -128,6 +128,7 @@
     clippy::shadow_unrelated,
     clippy::significant_drop_in_scrutinee,
     clippy::significant_drop_tightening,
+    clippy::single_option_map,
     clippy::str_to_string,
     clippy::string_add,
     clippy::string_lit_as_bytes,

src/metric.rs

@@ -122,7 +122,6 @@ impl<M> Fallible<M> {
     /// # Errors
     ///
     /// If this [`Fallible`] contains a [`prometheus::Error`].
-    #[expect(clippy::missing_const_for_fn, reason = "non-const deref coercion")]
     pub fn as_ref(&self) -> Result<&Arc<Metric<M>>, &prometheus::Error> {
         (*self.0).as_ref()
     }