Skip to content

ci: name compiler binaries in *-compiler-binary; drop forwarded-deps-outputs - #369

Merged
mcocdawc merged 2 commits into
developfrom
drop-build-action-fields
Oct 5, 2026
Merged

mcocdawc merged 2 commits into
developfrom
drop-build-action-fields

Conversation

@mcocdawc

@mcocdawc mcocdawc commented Oct 5, 2026 •

Copy link
Copy Markdown
Collaborator
  • HPC leg: cc/cxx become c-compiler-binary/cxx-compiler-binary.
  • forwarded-deps-outputs goes: it only fed a build action, and build-hpc runs a job-script.

Rendered scripts are unchanged for every existing leg, and so are the generated cross-repo-trigger*.yml; the regenerate bot brings trigger-downstream*.yml up to date. Needed before ecmwf/ci-infrastructure#103, which removes these fields; it works on ci-infrastructure main today.

🤖 Generated with Claude Code

🌦️ >> Documentation << 🌦️
https://sites.ecmwf.int/docs/dev-section/eckit/pull-requests/PR-369

…outputs

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@codecov-commenter

codecov-commenter commented Oct 5, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 68.75%. Comparing base (ee09656) to head (2490f3f).
⚠️ Report is 3 commits behind head on develop.

Additional details and impacted files
@@           Coverage Diff            @@
##           develop     #369   +/-   ##
========================================
  Coverage    68.74%   68.75%           
========================================
  Files         1189     1189           
  Lines        63064    63064           
  Branches      4748     4748           
========================================
+ Hits         43355    43361    +6     
+ Misses       19709    19703    -6     

☔ 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.

@tbkr tbkr left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Much welcomed change, thanks a lot for taking the time and implementing this much requested feature.

I feel this will heavily improve the overall quality and make process more frictionless.

Thanks a lot.

LGTM.

@mcocdawc
mcocdawc merged commit 470c223 into develop Oct 5, 2026
92 of 93 checks passed
@mcocdawc
mcocdawc deleted the drop-build-action-fields branch October 5, 2026 14:07
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.

3 participants