Merge pull request #55 from hiro-o918/fix/clippy

🔨 make ci fail on clippy warning
hiro-o918 · Jul 30, 2022 · 25b724e · 25b724e
2 parents 52058e2 + 7c77a3b
commit 25b724e
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 6 deletions.
diff --git a/.github/workflows/lint.yml b/.github/workflows/lint.yml
@@ -73,4 +73,4 @@ jobs:
       - uses: actions-rs/clippy-check@v1
         with:
           token: ${{ secrets.GITHUB_TOKEN }}
-          args: --all-features
+          args: --all-features -- -D warnings
diff --git a/src/creds.rs b/src/creds.rs
@@ -190,11 +190,7 @@ fn parse_aws_credentials(file: &File) -> Result<CredentialData, ctx::CTXError> {
         .context("failed to deserialize credentials".to_string())
         .map_or_else(
             |e| Err(ctx::CTXError::CredentialsIsBroken { source: Some(e) }),
-            |h| {
-                Ok(h.into_iter()
-                    .map(|(k, v)| (k.to_string(), Rc::new(v)))
-                    .collect())
-            },
+            |h| Ok(h.into_iter().map(|(k, v)| (k, Rc::new(v))).collect()),
         )
 }