Skip to content
Closed
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 3 additions & 1 deletion .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,9 @@ name: CI
on:
pull_request:
branches: ['**', '!changeset-release/**']
# CI only runs on PRs - merge queue validates before merge to main
push:
branches: [main]
# TODO: Optimize - separate deployment workflow to avoid re-running tests on main
Comment on lines +5 to +7

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P1 Badge Push to main triggers skip all work

The new push trigger for main means this workflow now runs on merges, but NX_BASE is still set to origin/main while NX_HEAD is HEAD. On a push event those refs are identical, so every nx affected invocation reports no affected projects and the build/test/deploy jobs effectively do nothing. As a result, pushes to main won’t rebuild or deploy the website even though the workflow runs.

Useful? React with 👍 / 👎.


concurrency:
group: ${{ github.workflow }}-${{ github.ref }}
Expand Down
Loading