Run clippy on all targets

This catches compile errors in the benchmarks.

Author: jyn514

.github/workflows/ci.yml

@@ -134,7 +134,7 @@ jobs:
       - uses: actions-rs/cargo@v1
         with:
           command: clippy
-          args: --workspace --locked -- -D warnings
+          args: --all-targets --workspace --locked -- -D warnings
 
       - name: Clean unused artifacts
         uses: actions-rs/cargo@v1

src/test/fakes.rs

@@ -230,8 +230,8 @@ impl<'a> FakeRelease<'a> {
         // Upload all source files as rustdoc files
         // In real life, these would be highlighted HTML, but for testing we just use the files themselves.
         for (source_path, data) in &self.source_files {
-            if source_path.starts_with("src/") {
-                let updated = ["src", &package.name, &source_path[4..]].join("/");
+            if let Some(src) = source_path.strip_prefix("src/") {
+                let updated = ["src", &package.name, src].join("/");
                 rustdoc_files.push((Box::leak(Box::new(updated)), data));
             }
         }

src/web/mod.rs

@@ -715,11 +715,11 @@ mod test {
                 assert_redirect(&format!("/{}", krate), &target, web)?;
                 assert_redirect(&format!("/{}/", krate), &target, web)?;
             }
-            let target = format!("https://doc.rust-lang.org/stable/proc_macro/");
 
+            let target = "https://doc.rust-lang.org/stable/proc_macro/";
             // with or without slash
-            assert_redirect("/proc-macro", &target, web)?;
-            assert_redirect("/proc-macro/", &target, web)?;
+            assert_redirect("/proc-macro", target, web)?;
+            assert_redirect("/proc-macro/", target, web)?;
             Ok(())
         })
     }