Move lint step after build, for protoc.

skymethod · Dec 4, 2023 · 32c7903 · 32c7903
1 parent 889858b
commit 32c7903
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/.github/workflows/npm.yml b/.github/workflows/npm.yml
@@ -89,9 +89,6 @@ jobs:
       - name: Check formatting
         run: cargo fmt -- --check
         working-directory: ./napi
-      - name: Check linting
-        run: cargo clippy --release --all-targets --all-features -- -D clippy::all
-        working-directory: ./napi
       - name: Setup node x86
         if: matrix.settings.target == 'i686-pc-windows-msvc'
         run: yarn config set supportedArchitectures.cpu "ia32"
@@ -120,6 +117,9 @@ jobs:
         if: ${{ !matrix.settings.docker }}
         shell: bash
         working-directory: ./napi
+      - name: Check linting # needs to run after build so protoc is available
+        run: cargo clippy --release --all-targets --all-features -- -D clippy::all
+        working-directory: ./napi
       - name: Upload artifact
         uses: actions/upload-artifact@v3
         with: