From 8a8b29ea4e73ab1db01308dc6b05dca98959ca09 Mon Sep 17 00:00:00 2001 From: Francesco Novy Date: Wed, 9 Nov 2022 12:55:18 +0100 Subject: [PATCH] chore(ci): Always run Next.js & Node SDK tests As these are currently required to merge PRs. We should figure out a better solution for this, but for now this unblocks merging PRs. --- .github/workflows/build.yml | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml index 68342d226548..88568ab6e080 100644 --- a/.github/workflows/build.yml +++ b/.github/workflows/build.yml @@ -402,7 +402,7 @@ jobs: job_nextjs_integration_test: name: Test @sentry/nextjs on (Node ${{ matrix.node }}) needs: [job_get_metadata, job_build] - if: needs.job_get_metadata.outputs.changed_nextjs == 'true' || github.event_name != 'pull_request' + # Currently always runs because it is required for merging PRs continue-on-error: true timeout-minutes: 30 runs-on: ubuntu-latest @@ -608,7 +608,7 @@ jobs: job_node_integration_tests: name: Node SDK Integration Tests (${{ matrix.node }}) needs: [job_get_metadata, job_build] - if: needs.job_get_metadata.outputs.changed_node == 'true' || github.event_name != 'pull_request' + # Currently always runs because it is required for merging PRs runs-on: ubuntu-latest timeout-minutes: 10 continue-on-error: true