style: Enable template lints

Author: epage

Cargo.toml

@@ -18,72 +18,72 @@ include = [
 ]
 
 [workspace.lints.rust]
-# rust_2018_idioms = { level = "warn", priority = -1 }
-# unreachable_pub = "warn"
+rust_2018_idioms = { level = "warn", priority = -1 }
+unreachable_pub = "warn"
 unsafe_op_in_unsafe_fn = "warn"
 unused_lifetimes = "warn"
 unused_macro_rules = "warn"
-# unused_qualifications = "warn"
+unused_qualifications = "warn"
 
 [workspace.lints.clippy]
 bool_assert_comparison = "allow"
 branches_sharing_code = "allow"
-# checked_conversions = "warn"
+checked_conversions = "warn"
 collapsible_else_if = "allow"
-# create_dir = "warn"
-# dbg_macro = "warn"
-# debug_assert_with_mut_call = "warn"
-# doc_markdown = "warn"
-# empty_enum = "warn"
-# enum_glob_use = "warn"
-# expl_impl_clone_on_copy = "warn"
-# explicit_deref_methods = "warn"
-# explicit_into_iter_loop = "warn"
-# fallible_impl_from = "warn"
-# filter_map_next = "warn"
-# flat_map_option = "warn"
-# float_cmp_const = "warn"
-# fn_params_excessive_bools = "warn"
-# from_iter_instead_of_collect = "warn"
+create_dir = "warn"
+dbg_macro = "warn"
+debug_assert_with_mut_call = "warn"
+doc_markdown = "warn"
+empty_enum = "warn"
+enum_glob_use = "warn"
+expl_impl_clone_on_copy = "warn"
+explicit_deref_methods = "warn"
+explicit_into_iter_loop = "warn"
+fallible_impl_from = "warn"
+filter_map_next = "warn"
+flat_map_option = "warn"
+float_cmp_const = "warn"
+fn_params_excessive_bools = "warn"
+from_iter_instead_of_collect = "warn"
 if_same_then_else = "allow"
-# implicit_clone = "warn"
-# imprecise_flops = "warn"
-# inconsistent_struct_constructor = "warn"
-# inefficient_to_string = "warn"
-# infinite_loop = "warn"
-# invalid_upcast_comparisons = "warn"
-# large_digit_groups = "warn"
-# large_stack_arrays = "warn"
-# large_types_passed_by_value = "warn"
+implicit_clone = "warn"
+imprecise_flops = "warn"
+inconsistent_struct_constructor = "warn"
+inefficient_to_string = "warn"
+infinite_loop = "warn"
+invalid_upcast_comparisons = "warn"
+large_digit_groups = "warn"
+large_stack_arrays = "warn"
+large_types_passed_by_value = "warn"
 let_and_return = "allow"  # sometimes good to name what you are returning
-# linkedlist = "warn"
-# lossy_float_literal = "warn"
-# macro_use_imports = "warn"
-# mem_forget = "warn"
-# mutex_integer = "warn"
-# needless_continue = "warn"
-# needless_for_each = "warn"
-# negative_feature_names = "warn"
-# path_buf_push_overwrite = "warn"
-# ptr_as_ptr = "warn"
-# rc_mutex = "warn"
-# redundant_feature_names = "warn"
-# ref_option_ref = "warn"
-# rest_pat_in_fully_bound_structs = "warn"
-# same_functions_in_if_condition = "warn"
-# self_named_module_files = "warn"
-# semicolon_if_nothing_returned = "warn"
-# str_to_string = "warn"
-# string_add = "warn"
-# string_add_assign = "warn"
-# string_lit_as_bytes = "warn"
-# string_to_string = "warn"
-# todo = "warn"
-# trait_duplication_in_bounds = "warn"
-# uninlined_format_args = "warn"
-# verbose_file_reads = "warn"
-# wildcard_imports = "warn"
-# zero_sized_map_values = "warn"
+linkedlist = "warn"
+lossy_float_literal = "warn"
+macro_use_imports = "warn"
+mem_forget = "warn"
+mutex_integer = "warn"
+needless_continue = "warn"
+needless_for_each = "warn"
+negative_feature_names = "warn"
+path_buf_push_overwrite = "warn"
+ptr_as_ptr = "warn"
+rc_mutex = "warn"
+redundant_feature_names = "warn"
+ref_option_ref = "warn"
+rest_pat_in_fully_bound_structs = "warn"
+same_functions_in_if_condition = "warn"
+self_named_module_files = "warn"
+semicolon_if_nothing_returned = "warn"
+str_to_string = "warn"
+string_add = "warn"
+string_add_assign = "warn"
+string_lit_as_bytes = "warn"
+string_to_string = "warn"
+todo = "warn"
+trait_duplication_in_bounds = "warn"
+uninlined_format_args = "warn"
+verbose_file_reads = "warn"
+wildcard_imports = "warn"
+zero_sized_map_values = "warn"
 
 [package]
 name = "config"

examples/custom_file_format/main.rs

@@ -23,7 +23,7 @@ fn main() {
 
     // Deserialize the config object into your Settings struct:
     let settings: Settings = settings.try_deserialize().unwrap();
-    println!("{:#?}", settings);
+    println!("{settings:#?}");
 }
 
 #[derive(Debug, Clone)]
@@ -34,7 +34,7 @@ impl Format for PemFile {
         &self,
         uri: Option<&String>,
         text: &str,
-    ) -> Result<Map<String, config::Value>, Box<dyn std::error::Error + Send + Sync>> {
+    ) -> Result<Map<String, Value>, Box<dyn std::error::Error + Send + Sync>> {
         // Store any valid keys into this map, they'll be merged with other sources into the final config map:
         let mut result = Map::new();
 

examples/custom_str_format/main.rs

@@ -7,8 +7,8 @@ fn main() {
         .build();
 
     match config {
-        Ok(cfg) => println!("A config: {:#?}", cfg),
-        Err(e) => println!("An error: {}", e),
+        Ok(cfg) => println!("A config: {cfg:#?}"),
+        Err(e) => println!("An error: {e}"),
     }
 }
 
@@ -20,7 +20,7 @@ impl Format for MyFormat {
         &self,
         uri: Option<&String>,
         text: &str,
-    ) -> Result<Map<String, config::Value>, Box<dyn std::error::Error + Send + Sync>> {
+    ) -> Result<Map<String, Value>, Box<dyn std::error::Error + Send + Sync>> {
         // Let's assume our format is somewhat malformed, but this is fine
         // In real life anything can be used here - nom, serde or other.
         //
@@ -29,11 +29,11 @@ impl Format for MyFormat {
 
         if text == "good" {
             result.insert(
-                "key".to_string(),
+                "key".to_owned(),
                 Value::new(uri, ValueKind::String(text.into())),
             );
         } else {
-            println!("Something went wrong in {:?}", uri);
+            println!("Something went wrong in {uri:?}");
         }
 
         Ok(result)

examples/hierarchical-env/main.rs

@@ -6,5 +6,5 @@ fn main() {
     let settings = Settings::new();
 
     // Print out our settings
-    println!("{:?}", settings);
+    println!("{settings:?}");
 }

examples/hierarchical-env/settings.rs

@@ -34,7 +34,7 @@ struct Braintree {
 
 #[derive(Debug, Deserialize)]
 #[allow(unused)]
-pub struct Settings {
+pub(crate) struct Settings {
     debug: bool,
     database: Database,
     sparkpost: Sparkpost,
@@ -43,7 +43,7 @@ pub struct Settings {
 }
 
 impl Settings {
-    pub fn new() -> Result<Self, ConfigError> {
+    pub(crate) fn new() -> Result<Self, ConfigError> {
         let run_mode = env::var("RUN_MODE").unwrap_or_else(|_| "development".into());
 
         let s = Config::builder()
@@ -53,7 +53,7 @@ impl Settings {
             // Default to 'development' env
             // Note that this file is _optional_
             .add_source(
-                File::with_name(&format!("examples/hierarchical-env/config/{}", run_mode))
+                File::with_name(&format!("examples/hierarchical-env/config/{run_mode}"))
                     .required(false),
             )
             // Add in a local configuration file

examples/watch/main.rs

@@ -28,7 +28,7 @@ fn show() {
     );
 }
 
-fn watch() {
+fn watch() -> ! {
     // Create a channel to receive the events.
     let (tx, rx) = channel();
 
@@ -62,7 +62,7 @@ fn watch() {
                 show();
             }
 
-            Err(e) => println!("watch error: {:?}", e),
+            Err(e) => println!("watch error: {e:?}"),
 
             _ => {
                 // Ignore event

src/de.rs

@@ -269,15 +269,15 @@ struct EnumAccess {
 }
 
 impl EnumAccess {
-    fn variant_deserializer(&self, name: &str) -> Result<StrDeserializer> {
+    fn variant_deserializer(&self, name: &str) -> Result<StrDeserializer<'_>> {
         self.variants
             .iter()
             .find(|&&s| s.to_lowercase() == name.to_lowercase()) // changing to lowercase will enable deserialization of lowercase values to enums
             .map(|&s| StrDeserializer(s))
             .ok_or_else(|| self.no_constructor_error(name))
     }
 
-    fn table_deserializer(&self, table: &Table) -> Result<StrDeserializer> {
+    fn table_deserializer(&self, table: &Table) -> Result<StrDeserializer<'_>> {
         if table.len() == 1 {
             self.variant_deserializer(table.iter().next().unwrap().0)
         } else {

src/env.rs

@@ -34,12 +34,12 @@ pub struct Environment {
 
     /// Optional directive to translate collected keys into a form that matches what serializers
     /// that the configuration would expect. For example if you have the `kebab-case` attribute
-    /// for your serde config types, you may want to pass Case::Kebab here.
+    /// for your serde config types, you may want to pass `Case::Kebab` here.
     #[cfg(feature = "convert-case")]
-    convert_case: Option<convert_case::Case>,
+    convert_case: Option<Case>,
 
-    /// Optional character sequence that separates each env value into a vector. only works when try_parsing is set to true
-    /// Once set, you cannot have type String on the same environment, unless you set list_parse_keys.
+    /// Optional character sequence that separates each env value into a vector. only works when `try_parsing` is set to true
+    /// Once set, you cannot have type String on the same environment, unless you set `list_parse_keys`.
     list_separator: Option<String>,
     /// A list of keys which should always be parsed as a list. If not set you can have only `Vec<String>` or `String` (not both) in one environment.
     list_parse_keys: Option<Vec<String>>,
@@ -110,7 +110,7 @@ impl Environment {
         }
     }
 
-    /// See [Environment::with_prefix]
+    /// See [`Environment::with_prefix`]
     pub fn prefix(mut self, s: &str) -> Self {
         self.prefix = Some(s.into());
         self
@@ -141,7 +141,7 @@ impl Environment {
         self
     }
 
-    /// When set and try_parsing is true, then all environment variables will be parsed as [`Vec<String>`] instead of [`String`].
+    /// When set and `try_parsing` is true, then all environment variables will be parsed as [`Vec<String>`] instead of [`String`].
     /// See
     /// [`with_list_parse_key`](Self::with_list_parse_key)
     /// when you want to use [`Vec<String>`] in combination with [`String`].
@@ -151,11 +151,11 @@ impl Environment {
     }
 
     /// Add a key which should be parsed as a list when collecting [`Value`]s from the environment.
-    /// Once list_separator is set, the type for string is [`Vec<String>`].
+    /// Once `list_separator` is set, the type for string is [`Vec<String>`].
     /// To switch the default type back to type Strings you need to provide the keys which should be [`Vec<String>`] using this function.
     pub fn with_list_parse_key(mut self, key: &str) -> Self {
         if self.list_parse_keys.is_none() {
-            self.list_parse_keys = Some(vec![key.to_lowercase()])
+            self.list_parse_keys = Some(vec![key.to_lowercase()]);
         } else {
             self.list_parse_keys = self.list_parse_keys.map(|mut keys| {
                 keys.push(key.to_lowercase());
@@ -246,7 +246,7 @@ impl Source for Environment {
         let prefix_pattern = self
             .prefix
             .as_ref()
-            .map(|prefix| format!("{}{}", prefix, prefix_separator).to_lowercase());
+            .map(|prefix| format!("{prefix}{prefix_separator}").to_lowercase());
 
         let collector = |(key, value): (String, String)| {
             // Treat empty environment variables as unset
@@ -295,7 +295,7 @@ impl Source for Environment {
                         if keys.contains(&key) {
                             let v: Vec<Value> = value
                                 .split(separator)
-                                .map(|s| Value::new(Some(&uri), ValueKind::String(s.to_string())))
+                                .map(|s| Value::new(Some(&uri), ValueKind::String(s.to_owned())))
                                 .collect();
                             ValueKind::Array(v)
                         } else {
@@ -304,7 +304,7 @@ impl Source for Environment {
                     } else {
                         let v: Vec<Value> = value
                             .split(separator)
-                            .map(|s| Value::new(Some(&uri), ValueKind::String(s.to_string())))
+                            .map(|s| Value::new(Some(&uri), ValueKind::String(s.to_owned())))
                             .collect();
                         ValueKind::Array(v)
                     }