Description

Originally, I thought #5429 would fix the issue of the npm job not running on merges into main. However, that didn't work. Turns out the action/workflow we were trying to fix is not a pull_request event but instead a push event.

image (1)

Therefore, we modified the logic to run on PRs into main (logic unchanged in file) and added logic to run on all other events besides pull_request. This protects it from running on PRs from forks but also allows it to run on push into main. We went with this approach instead of explicitly writing main in the logic so that it doesn't break if we change the default branch name.

Sources

• https://docs.github.com/en/developers/webhooks-and-events/webhooks/webhook-events-and-payloads#pull_request
• https://docs.github.com/en/actions/using-workflows/events-that-trigger-workflows#available-events
• https://docs.github.com/en/actions/using-workflows/events-that-trigger-workflows#push
• https://docs.github.com/en/actions/using-workflows/events-that-trigger-workflows#push
• https://docs.github.com/en/actions/learn-github-actions/contexts#github-context

Fixes N/A