chore(ci): better clippy lints, deny unwrap() outside of tests (#34)

Signed-off-by: Dori Medini <[email protected]>

Cargo.toml

@@ -28,3 +28,4 @@ unused = "deny"
 
 [workspace.lints.clippy]
 as_conversions = "deny"
+unwrap_used = "deny"

clippy.toml

@@ -0,0 +1 @@
+allow-unwrap-in-tests = true

crates/committer_cli/src/main.rs

@@ -56,7 +56,8 @@ fn main() {
             );
 
             // Business logic to be implemented here.
-            let output = std::fs::read(input_file_name).unwrap();
+            let output = std::fs::read(input_file_name)
+                .unwrap_or_else(|_| panic!("Failed to read input from file '{input_file_name:?}'"));
 
             // Output to file.
             std::fs::write(output_file_name, output).expect("Failed to write output");

scripts/clippy.sh

@@ -1,4 +1,3 @@
 #!/bin/bash
 
-cargo clippy "$@" --all-targets --all-features -- -D warnings -D future-incompatible \
-    -D nonstandard-style -D rust-2018-idioms -D unused
+cargo clippy "$@" --all-targets --all-features