run pre-commit within the same step after installing it; ci.yml now t…

…riggered by push only
drink7036290 · Dec 18, 2024 · 317cb1d · 317cb1d
1 parent 84f207b
commit 317cb1d
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 8 deletions.
diff --git a/.github/actions/basics/action.yml b/.github/actions/basics/action.yml
@@ -21,16 +21,13 @@ runs:
       with:
         node-version: "latest"
 
-    - name: Install pre-commit
+    # 4. Run pre-commit hooks on all files
+    - name: Run pre-commit
       run: |
         python3 -m venv venv
         source venv/bin/activate
         pip install pre-commit
-      shell: bash
-
-    # 4. Run pre-commit hooks on all files
-    - name: Run pre-commit
-      run: pre-commit run --all-files
+        pre-commit run --all-files
       shell: bash
 
     # 5. Run linkspector

diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -3,8 +3,6 @@ name: CI
 on:
   push:
     branches: ["main"]
-  pull_request:
-    branches: ["main"]
 
 env:
   CARGO_TERM_COLOR: always