fix: dedupe ci jobs

[agouin]

Fix integration and test jobs running twice per push on PRs.

[mark-rushakoff]

If I am following this right, opening a PR with commit A would be tested; then pushing commit B to the same branch (and therefore the same pull request) would not result in tests being run for commit B?

[agouin]

If I am following this right, opening a PR with commit A would be tested; then pushing commit B to the same branch (and therefore the same pull request) would not result in tests being run for commit B?

No, tests are run for every commit pushed to the PR still. This pattern is used in most of our other repos.

[mark-rushakoff]

Okay, I see now. That was surprisingly difficult to stumble across in the documentation.

Runs your workflow when activity on a pull request in the workflow's repository occurs. For example, if no activity types are specified, the workflow runs when a pull request is opened or reopened or when the head branch of the pull request is updated.

https://docs.github.com/en/actions/writing-workflows/choosing-when-your-workflow-runs/events-that-trigger-workflows#pull_request