optimize CI: on pushes to master, only run the docs step, skip others (#447)

Author: alindima

.github/scripts/matrices.py

@@ -70,7 +70,7 @@ def __init__(
 t_linux_x86 = Target("parity-large-new", "x86_64-unknown-linux-gnu", "linux-amd64")
 # TODO: Figure out how to make this work
 # t_linux_arm = Target("ubuntu-latest", "aarch64-unknown-linux-gnu", "linux-aarch64")
-t_macos = Target("parity-macos", "aarch64-apple-darwin", "macosx-aarch64")
+t_macos = Target("macos-latest", "aarch64-apple-darwin", "macosx-aarch64")
 t_windows = Target("windows-latest", "x86_64-pc-windows-msvc", "windows-amd64")
 targets = [t_linux_x86, t_macos, t_windows] if is_pr else [t_linux_x86, t_macos, t_windows]
 

.github/workflows/nextest.yml

@@ -8,7 +8,7 @@ on:
         type: string
 
 concurrency:
-  group: tests-${{ github.workflow }}-${{ github.head_ref || github.run_id }}
+  group: tests-${{ github.workflow }}-${{ github.head_ref || github.ref }}
   cancel-in-progress: true
 
 env:
@@ -38,7 +38,6 @@ jobs:
           PROFILE: ${{ inputs.profile }}
         run: |
           output=$(python3 .github/scripts/matrices.py)
-          echo $output
           echo "::debug::test-matrix=$output"
           echo "test-matrix=$output" >> $GITHUB_OUTPUT
 
@@ -53,7 +52,7 @@ jobs:
           - os: ubuntu
             runner: parity-large-new
           - os: macos
-            runner: parity-macos
+            runner: macos-latest
           - os: windows
             runner: windows-latest
     steps:
@@ -75,10 +74,6 @@ jobs:
           path: ~/.cargo/bin/eth-rpc${{ matrix.os == 'windows' && '.exe' || '' }}
           key: eth-rpc-${{ matrix.runner }}-${{ env.ETH_RPC_VERSION }}
 
-      - name: Set up Homebrew (for mac)
-        if: matrix.os == 'macos'
-        uses: Homebrew/actions/setup-homebrew@1ccc07ccd54b6048295516a3eb89b192c35057dc
-
       # Install dependencies for building
       - name: Install clang on ubuntu
         if: matrix.os == 'ubuntu'
@@ -154,14 +149,6 @@ jobs:
       ETH_RPC_URL: https://eth-mainnet.alchemyapi.io/v2/${{ secrets.ALCHEMY_API_KEY }}
       CARGO_PROFILE_DEV_DEBUG: 0
     steps:
-      - name: Set up Homebrew (for mac)
-        if: contains(matrix.runner_label, 'macos')
-        uses: Homebrew/actions/setup-homebrew@1ccc07ccd54b6048295516a3eb89b192c35057dc
-      - name: Install curl and jq
-        if: contains(matrix.runner_label, 'macos')
-        run: |
-          brew install curl jq
-
       - uses: actions/checkout@v4
       - uses: taiki-e/install-action@nextest
 

.github/workflows/test.yml

@@ -7,7 +7,7 @@ on:
   pull_request:
 
 concurrency:
-  group: ${{ github.workflow }}-${{ github.head_ref || github.run_id }}
+  group: ${{ github.workflow }}-${{ github.head_ref || github.ref }}
   cancel-in-progress: true
 
 env:
@@ -16,6 +16,7 @@ env:
 
 jobs:
   nextest:
+    if: github.event_name == 'pull_request'
     uses: ./.github/workflows/nextest.yml
     with:
       profile: default
@@ -53,6 +54,7 @@ jobs:
           force_orphan: true
 
   doctest:
+    if: github.event_name == 'pull_request'
     runs-on: parity-large-new
     timeout-minutes: 60
     steps:
@@ -74,13 +76,15 @@ jobs:
       - run: cargo test --workspace --doc
 
   typos:
+    if: github.event_name == 'pull_request'
     runs-on: ubuntu-latest
     timeout-minutes: 30
     steps:
       - uses: actions/checkout@v4
       - uses: crate-ci/typos@v1
 
   clippy:
+    if: github.event_name == 'pull_request'
     runs-on: ubuntu-latest
     timeout-minutes: 60
     steps:
@@ -107,6 +111,7 @@ jobs:
           RUSTFLAGS: -Dwarnings
 
   rustfmt:
+    if: github.event_name == 'pull_request'
     runs-on: ubuntu-latest
     timeout-minutes: 30
     steps:
@@ -117,6 +122,7 @@ jobs:
       - run: cargo fmt --all --check
 
   forge-fmt:
+    if: github.event_name == 'pull_request'
     runs-on: ubuntu-latest
     timeout-minutes: 30
     steps:
@@ -135,6 +141,7 @@ jobs:
         run: ./.github/scripts/format.sh --check
 
   crate-checks:
+    if: github.event_name == 'pull_request'
     runs-on: parity-large-new
     timeout-minutes: 90
     steps:
@@ -167,6 +174,7 @@ jobs:
       - run: cargo hack check -p substrate-runtime --each-feature --exclude-no-default-features
 
   deny:
+    if: github.event_name == 'pull_request'
     # Copy of ithacaxyz/ci/.github/workflows/deny.yml@main but without failing the CI
     name: cargo deny check
     runs-on: ubuntu-latest
@@ -184,8 +192,8 @@ jobs:
         run: cargo deny --all-features check all
 
   ci-success:
+    if: github.event_name == 'pull_request'
     runs-on: ubuntu-latest
-    if: always()
     needs:
       - nextest
       - docs