Skip to content

Commit

Permalink
🔨 Fix CI for fork repos (#3115)
Browse files Browse the repository at this point in the history
* chore(ci): Attempt fix check-wasm-size for fork repos

* chore(ci): add permission (issues: write) to check-wasm-size

* chore(ci): disable check-wasm-size in PR from from fork repos

* chore(ci): improve job triggers

* chore(ci): fix tracing tests

* Revert "chore(ci): fix tracing tests"

This reverts commit ad0b03d.

* chore(ci): skip tracing tests for PR's originated from forked repos
  • Loading branch information
RomarQ authored Dec 20, 2024
1 parent 7dbae97 commit ef2aade
Showing 1 changed file with 5 additions and 5 deletions.
10 changes: 5 additions & 5 deletions .github/workflows/build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -299,7 +299,7 @@ jobs:
permissions:
contents: read
pull-requests: write
if: github.event_name == 'pull_request'
if: ${{ github.event_name == 'pull_request' && !github.event.pull_request.head.repo.fork }}
needs: ["set-tags", "build"]
env:
GH_TOKEN: ${{ github.token }}
Expand Down Expand Up @@ -610,7 +610,7 @@ jobs:

typescript-tracing-tests:
if: >
(github.event_name == 'pull_request' && github.event.pull_request.head.repo.full_name == github.repository) ||
(github.event_name == 'pull_request' && !github.event.pull_request.head.repo.fork) ||
(github.event_name == 'push' && github.ref == 'refs/heads/master')
runs-on:
labels: bare-metal
Expand Down Expand Up @@ -720,7 +720,7 @@ jobs:
labels: bare-metal
permissions:
contents: read
needs: ["set-tags", "build", "typescript-tracing-tests"]
needs: ["set-tags", "build"]
strategy:
fail-fast: false
max-parallel: 1
Expand Down Expand Up @@ -778,7 +778,7 @@ jobs:
chopsticks-upgrade-test:
runs-on:
labels: bare-metal
needs: ["set-tags", "build", "typescript-tracing-tests"]
needs: ["set-tags", "build"]
strategy:
fail-fast: false
matrix:
Expand Down Expand Up @@ -825,7 +825,7 @@ jobs:
zombie_upgrade_test:
runs-on:
labels: bare-metal
needs: ["set-tags", "build", "typescript-tracing-tests"]
needs: ["set-tags", "build"]
strategy:
fail-fast: false
max-parallel: 1
Expand Down

0 comments on commit ef2aade

Please sign in to comment.