From e3c57fa83d9734e91a67e6505bdb4405acecaca1 Mon Sep 17 00:00:00 2001
From: Charles Hall <charles@computer.surgery>
Date: Tue, 14 May 2024 19:50:07 -0700
Subject: [PATCH] enable most pedantic lints as a group

I'm turning off the documentation related ones because they generate
way too many warnings, this kind of thing will need to be improved over
a longer timespan.
---
 Cargo.toml | 42 ++++++++----------------------------------
 1 file changed, 8 insertions(+), 34 deletions(-)

diff --git a/Cargo.toml b/Cargo.toml
index 9509250c..671a4e6a 100644
--- a/Cargo.toml
+++ b/Cargo.toml
@@ -15,88 +15,62 @@ unused_lifetimes = "warn"
 unused_macro_rules = "warn"
 unused_qualifications = "warn"
 
-# Keep alphabetically sorted
 [workspace.lints.clippy]
+# Groups. Keep alphabetically sorted
+pedantic = "warn"
+
+# Lints. Keep alphabetically sorted
 as_conversions = "warn"
 assertions_on_result_states = "warn"
-cloned_instead_of_copied = "warn"
 dbg_macro = "warn"
-default_trait_access = "warn"
 default_union_representation = "warn"
 deref_by_slicing = "warn"
-doc_markdown = "warn"
 empty_drop = "warn"
 empty_structs_with_brackets = "warn"
-enum_glob_use = "warn"
 error_impl_error = "warn"
-explicit_into_iter_loop = "warn"
 filetype_is_file = "warn"
-flat_map_option = "warn"
 float_cmp_const = "warn"
 format_push_string = "warn"
 get_unwrap = "warn"
-if_not_else = "warn"
 if_then_some_else_none = "warn"
-ignored_unit_patterns = "warn"
 impl_trait_in_params = "warn"
-implicit_clone = "warn"
-inconsistent_struct_constructor = "warn"
-items_after_statements = "warn"
 let_underscore_must_use = "warn"
 lossy_float_literal = "warn"
-manual_let_else = "warn"
-manual_string_new = "warn"
-map_unwrap_or = "warn"
-match_bool = "warn"
-match_same_arms = "warn"
-match_wildcard_for_single_variants = "warn"
 mem_forget = "warn"
 missing_assert_message = "warn"
 mod_module_files = "warn"
 multiple_inherent_impl = "warn"
 mutex_atomic = "warn"
-needless_pass_by_value = "warn"
 negative_feature_names = "warn"
 pub_without_shorthand = "warn"
 rc_buffer = "warn"
 rc_mutex = "warn"
-redundant_closure_for_method_calls = "warn"
 redundant_feature_names = "warn"
 redundant_type_annotations = "warn"
 ref_patterns = "warn"
 rest_pat_in_fully_bound_structs = "warn"
 same_name_method = "warn"
-semicolon_if_nothing_returned = "warn"
 semicolon_inside_block = "warn"
-similar_names = "warn"
-single_match_else = "warn"
 str_to_string = "warn"
 string_add = "warn"
 string_lit_chars_any = "warn"
 string_slice = "warn"
 string_to_string = "warn"
-struct_excessive_bools = "warn"
 suspicious_xor_used_as_pow = "warn"
 tests_outside_test_module = "warn"
-too_many_lines = "warn"
 try_err = "warn"
 undocumented_unsafe_blocks = "warn"
-uninlined_format_args = "warn"
 unnecessary_safety_comment = "warn"
 unnecessary_safety_doc = "warn"
 unnecessary_self_imports = "warn"
-unnecessary_wraps = "warn"
 unneeded_field_pattern = "warn"
-unnested_or_patterns = "warn"
-unreadable_literal = "warn"
 unseparated_literal_suffix = "warn"
-unused_async = "warn"
-unused_self = "warn"
-used_underscore_binding = "warn"
 verbose_file_reads = "warn"
 wildcard_dependencies = "warn"
-wildcard_imports = "warn"
-zero_sized_map_values = "warn"
+
+# TODO: Remove these:
+missing_errors_doc = "allow"
+missing_panics_doc = "allow"
 
 [package]
 name = "grapevine"