ci: prevent duplicate runs on PRs, add Java 25 to matrix#160
Conversation
Agent-Logs-Url: https://github.com/danielflower/multi-module-maven-release-plugin/sessions/8ade552e-bd68-47c1-86b0-3b41e9229b5d Co-authored-by: danielflower <1056441+danielflower@users.noreply.github.com>
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 5f4adcf5eb
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| push: | ||
| branches: | ||
| - master |
There was a problem hiding this comment.
Trigger CI on the repository's default branch
Limiting push to master means this workflow will not run for pushes/merges on repositories whose active branch is main (which is the branch present in this repo), so post-merge CI and any direct-push validation are silently skipped. This change prevents duplicate PR runs, but in a main-based repo it also removes the only push-event coverage for the default branch.
Useful? React with 👍 / 👎.
CI was triggering twice for pull requests — once for the
pushevent on the PR branch and once for thepull_requestevent.Changes
pushtrigger tomasteronly — PR branches now only trigger CI viapull_request, eliminating the duplicate run[8, 11, 21, 25]