Skip to content

Commit a99d894

Browse files
authored
chore(ci): Always run Next.js & Node SDK tests (#6166)
As these are currently required to merge PRs. We should figure out a better solution for this, but for now this unblocks merging PRs.
1 parent 8214839 commit a99d894

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

.github/workflows/build.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -402,7 +402,7 @@ jobs:
402402
job_nextjs_integration_test:
403403
name: Test @sentry/nextjs on (Node ${{ matrix.node }})
404404
needs: [job_get_metadata, job_build]
405-
if: needs.job_get_metadata.outputs.changed_nextjs == 'true' || github.event_name != 'pull_request'
405+
# Currently always runs because it is required for merging PRs
406406
continue-on-error: true
407407
timeout-minutes: 30
408408
runs-on: ubuntu-latest
@@ -608,7 +608,7 @@ jobs:
608608
job_node_integration_tests:
609609
name: Node SDK Integration Tests (${{ matrix.node }})
610610
needs: [job_get_metadata, job_build]
611-
if: needs.job_get_metadata.outputs.changed_node == 'true' || github.event_name != 'pull_request'
611+
# Currently always runs because it is required for merging PRs
612612
runs-on: ubuntu-latest
613613
timeout-minutes: 10
614614
continue-on-error: true

0 commit comments

Comments
 (0)