Skip to content

Conversation

@paulRbr
Copy link
Member

@paulRbr paulRbr commented Jan 13, 2026

This commit adapts the diff to be able to read the overlays of the
previous version (the one that we take from git on the base
branch). We then have two list of overlay files (the previous overlay
files and the current overlay files), materialized with the new
variables overlays1 and overlays2.

We then pass this to the core diff function (which is modified in
bump-sh/cli#776 and thus needs the action to
update the bump-cli package to include that change).

@paulRbr paulRbr self-assigned this Jan 13, 2026
Copy link
Contributor

@fbraure fbraure left a comment

Choose a reason for hiding this comment

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

LGTM

@paulRbr paulRbr force-pushed the diff-overlays branch 3 times, most recently from 871b63b to 41557d5 Compare January 13, 2026 16:39
This commit adapts the diff to be able to read the overlays of the
_previous_ version (the one that we take from git on the base
branch). We then have two list of overlay files (the previous overlay
files and the current overlay files), materialized with the new
variables `overlays1` and `overlays2`.

We then pass this to the core diff function (which is modified in
bump-sh/cli#776 and thus needs the action to
update the bump-cli package to include that change).
@paulRbr paulRbr merged commit 2dc0ee0 into bump-sh:main Jan 13, 2026
4 checks passed
@paulRbr paulRbr deleted the diff-overlays branch January 13, 2026 16:45
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.

2 participants