Skip to content

ci: run development build once per pull request - #320

Merged
emaarco merged 1 commit into
developfrom
ci/dedupe-pr-builds
Oct 6, 2026
Merged

emaarco merged 1 commit into
developfrom
ci/dedupe-pr-builds

Conversation

@emaarco

@emaarco emaarco commented Oct 6, 2026

Copy link
Copy Markdown
Member

Summary

Every push to a branch with an open pull request started the Development branches workflow twice — once for push, once for pull_request — running the identical Maven build.

Changes

  • Restrict the push trigger to develop. Pull requests are still built via pull_request, and develop is still built after each merge.
  • Drop the github.event_name == 'push' condition from the Codecov upload step so coverage is still reported for pull requests (previously it came from the branch push run). Dependabot stays excluded because its runs cannot read CODECOV_TOKEN.

Notes

  • The job name Build and run tests is unchanged, so a required status check bound to the pull_request run keeps working.
  • Branches without an open pull request are no longer built automatically.

Restrict the push trigger to develop. Branches with an open pull request were built twice, once for push and once for pull_request.
@codecov

codecov Bot commented Oct 6, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 50.65%. Comparing base (07bf056) to head (c0ec150).

Additional details and impacted files
@@            Coverage Diff             @@
##             develop     #320   +/-   ##
==========================================
  Coverage      50.65%   50.65%           
  Complexity        80       80           
==========================================
  Files             40       40           
  Lines            458      458           
  Branches           9        9           
==========================================
  Hits             232      232           
  Misses           221      221           
  Partials           5        5           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@emaarco
emaarco merged commit a9ac2f0 into develop Oct 6, 2026
5 checks passed
@emaarco
emaarco deleted the ci/dedupe-pr-builds branch October 6, 2026 10:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant