fix: remove short tests step due to no caching

smartcontractkit · Jul 9, 2024 · 6c53b75 · 6c53b75
1 parent 94ed03d
commit 6c53b75
Showing 1 changed file with 1 addition and 6 deletions.
diff --git a/.github/workflows/ci-core.yml b/.github/workflows/ci-core.yml
@@ -91,7 +91,7 @@ jobs:
             os: ubuntu-latest-32cores-128GB
           - cmd: go_core_race_tests
             id: core_race
-            # use 64cores for overnight runs only
+            # use 64cores for overnight runs only due to massive number of runs from PRs
             os: ${{ github.event_name == 'schedule' && 'ubuntu-latest-64cores-256GB' || 'ubuntu-latest-32cores-128GB' }}
           - cmd: go_core_fuzz
             id: core_fuzz
@@ -115,11 +115,6 @@ jobs:
       - name: Setup Go
         if: ${{ needs.filter.outputs.changes == 'true' }}
         uses: ./.github/actions/setup-go
-        # Run short-tests first to short-circuit failures for `go_core_tests`
-        # These tests would be run below anyways if they didn't run here (cached)
-      - name: Run short tests
-        if: ${{ needs.filter.outputs.changes == 'true' && matrix.type.cmd == 'go_core_tests' }}
-        run: go test -short ./...
       - name: Setup Solana
         if: ${{ needs.filter.outputs.changes == 'true' }}
         uses: ./.github/actions/setup-solana