chore(ci): Enforce clippy for all targets (#6299)

.github/workflows/CI.yml

@@ -38,13 +38,15 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v3
+        with:
+          submodules: true
 
       - uses: actions-rs/toolchain@v1
         with:
           profile: minimal
           components: clippy
 
-      - run: cargo clippy --all -- -D warnings
+      - run: cargo clippy --all --all-targets -- -D warnings
 
   cargo-crev:
     name: Cargo crev

crates/swc_css_modules/tests/fixture.rs

@@ -43,7 +43,7 @@ fn imports(input: PathBuf) {
 
 #[testing::fixture("tests/fixture/**/*.css", exclude("compiled\\.css"))]
 fn compile(input: PathBuf) {
-    testing::run_test(false, |cm, handler| {
+    testing::run_test(false, |cm, _| {
         let fm = cm.load_file(&input).unwrap();
         let mut errors = vec![];
         let mut ss = swc_css_parser::parse_file(

crates/swc_ecma_codegen/benches/bench.rs

@@ -114,7 +114,7 @@ fn bench_emitter(b: &mut Bencher, s: &str) {
                 let _ = emitter.emit_module(&module);
             }
             black_box(buf);
-            let srcmap = cm.build_source_map(&mut src_map_buf);
+            let srcmap = cm.build_source_map(&src_map_buf);
             black_box(srcmap);
         });
         Ok(())

crates/swc_ecma_codegen/benches/with_parse.rs

@@ -113,7 +113,7 @@ fn bench_emitter(b: &mut Bencher, s: &str) {
                 let _ = emitter.emit_module(&module);
             }
             black_box(buf);
-            let srcmap = cm.build_source_map(&mut src_map_buf);
+            let srcmap = cm.build_source_map(&src_map_buf);
             black_box(srcmap);
         });
         Ok(())

crates/swc_ecma_codegen/examples/sourcemap.rs

@@ -45,7 +45,7 @@ fn parse_and_gen(entry: &Path) {
             emitter.emit_module(&m).unwrap();
         }
 
-        let srcmap = cm.build_source_map(&mut srcmap);
+        let srcmap = cm.build_source_map(&srcmap);
 
         fs::write("output.js", &code).unwrap();
 

crates/swc_ecma_codegen/tests/sourcemap.rs

@@ -352,7 +352,7 @@ fn identity(entry: PathBuf) {
         }
 
         let actual_code = String::from_utf8(wr).unwrap();
-        let actual_map = cm.build_source_map_with_config(&mut src_map, None, SourceMapConfigImpl);
+        let actual_map = cm.build_source_map_with_config(&src_map, None, SourceMapConfigImpl);
 
         let visualizer_url_for_actual = visualizer_url(&actual_code, &actual_map);