Deny warnings on CI

With apologies to @zeegomo for originally requesting the change.

Author: jyn514

.github/workflows/ci.yml

@@ -31,7 +31,7 @@ jobs:
         run: cargo build --locked
 
       - name: Run clippy
-        run: cargo clippy -- -D clippy::all -D unused-imports
+        run: cargo clippy -- -D warnings
 
       - name: Prepare the test environment
         run: |

src/web/mod.rs

@@ -209,10 +209,6 @@ impl MatchVersion {
             MatchVersion::None => None,
         }
     }
-
-    fn strip_id(self) -> Option<String> {
-        self.into_option().map(|(version, _id)| version)
-    }
 }
 
 /// Checks the database for crate releases that match the given name and version.
@@ -522,6 +518,12 @@ mod test {
     use html5ever::tendril::TendrilSink;
     use crate::web::{match_version, MatchVersion};
 
+    impl MatchVersion {
+        fn strip_id(self) -> Option<String> {
+            self.into_option().map(|(version, _id)| version)
+        }
+    }
+
     const DEFAULT_ID: i32 = 0;
 
     fn release(version: &str, db: &TestDatabase) -> i32 {