Skip to content

Bundle merge updated - #47

Merged
wdeconinck merged 4 commits into
ecmwf:mainfrom
pardallio:bundle-merge-updated
Jul 28, 2026
Merged

Bundle merge updated#47
wdeconinck merged 4 commits into
ecmwf:mainfrom
pardallio:bundle-merge-updated

Conversation

@pardallio

Copy link
Copy Markdown
Contributor

Description

fixed issue with -o argument, (needed to add --output so it could be passed into the config object)

added erase feature to erase sections of the bundle. if they have an erase: true entry

for example, to remove eccodes from the bundle, add in the update:

    - eccodes :
        erase: true

the resulting bundle will no have an eccodes section.

A similar outcome would be possible by setting

    - eccodes :
        dir: ./
        bundle: false

but it would pollute the bundle folder with an unnecessary eccodes symlink

Contributor Declaration

By opening this pull request, I affirm the following:

  • All authors agree to the Contributor License Agreement.
  • The code follows the project's coding standards.
  • I have performed self-review and added comments where needed.
  • I have added or updated tests to verify that my changes are effective and functional.
  • I have run all existing tests and confirmed they pass.

@codecov-commenter

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 49.19%. Comparing base (28d4d49) to head (d2cb0fc).

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #47      +/-   ##
==========================================
+ Coverage   49.12%   49.19%   +0.06%     
==========================================
  Files          51       51              
  Lines       10367    10380      +13     
==========================================
+ Hits         5093     5106      +13     
  Misses       5274     5274              

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

@wdeconinck wdeconinck left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks @pardallio that looks great!

@wdeconinck
wdeconinck merged commit 16f2d2b into ecmwf:main Jul 28, 2026
4 checks passed
wdeconinck pushed a commit that referenced this pull request Jul 28, 2026
Adds `erase` attribute to erase sections of the bundle.
wdeconinck added a commit that referenced this pull request Jul 28, 2026
* release/2.5.0:
  Version 2.5.0
  ci: migrate ecmwf-actions references to ecmwf (#42)
  Updates to ecbundle-merge (#47)
  Add "ecbundle merge" functionality (#46)
  Add --arch-dir argument to point to specific architecture directories (#43)
@wdeconinck

Copy link
Copy Markdown
Member

This and your previous PRs are now present in ecbundle 2.5.0

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