Make clippy deny warnings (#22)

awslabs · Nov 9, 2023 · 60e88ec · 60e88ec
1 parent 18b3a77
commit 60e88ec
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 7 deletions.
diff --git a/.github/workflows/build_and_test.yml b/.github/workflows/build_and_test.yml
@@ -27,7 +27,7 @@ jobs:
       - run: cargo fmt --all --check
       - run: RUSTFLAGS="-D warnings -F unsafe-code" cargo build --verbose
       - run: cargo doc --all-features --no-deps
-      - run: cargo clippy --all-features
+      - run: RUSTFLAGS="-D warnings" cargo clippy --all-features --tests
       - run: cargo test --verbose
       - run: cargo test --verbose --no-default-features
       - run: cargo bench --no-run

diff --git a/scripts/build_and_test.sh b/scripts/build_and_test.sh
@@ -11,8 +11,8 @@ RUSTFLAGS="-D warnings -F unsafe-code" cargo build --verbose && \
 echo ">>>> cargo doc --all-features --no-deps" && \
 cargo doc --all-features --no-deps && \
 
-echo ">>>> cargo clippy --all-features" && \
-cargo clippy --all-features && \
+echo ">>>> cargo clippy --all-features --tests" && \
+RUSTFLAGS="-D warnings" cargo clippy --all-features --tests && \
 
 echo ">>>> cargo test --verbose" && \
 cargo test --verbose && \

diff --git a/src/private/sources/retry.rs b/src/private/sources/retry.rs
@@ -17,10 +17,10 @@ pub struct BackoffStrategy {
 
 impl BackoffStrategy {
     pub(crate) fn get_backoff(&self) -> ExponentialBackoff {
-        let mut exponential_backoff = ExponentialBackoff::default();
-        exponential_backoff.max_elapsed_time =
-            Option::from(Duration::from_secs(self.time_limit_seconds));
-        exponential_backoff
+        ExponentialBackoff {
+            max_elapsed_time: Option::from(Duration::from_secs(self.time_limit_seconds)),
+            ..Default::default()
+        }
     }
 }