Skip to content

Conversation

paulRbr
Copy link
Member

@paulRbr paulRbr commented Jul 17, 2025

This commit is a fix to be able to keep track of references ($ref)
used inside an overlay. This make it possible to use an overlay which
references external files, and have them available for the
main (overlayed) definition.

Thanks @JakeSCahill for noticing the bug!

@paulRbr paulRbr force-pushed the dont-resolve-refs-on-overlay-files branch 3 times, most recently from cff9e38 to 291aa2f Compare July 17, 2025 13:50
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,
As you suggested, we can add a uniqness on external references sent

@paulRbr paulRbr force-pushed the dont-resolve-refs-on-overlay-files branch 2 times, most recently from 4413d4b to 28b984b Compare July 17, 2025 14:45
This commit is a fix to be able to keep track of references ($ref)
used inside an overlay. This make it possible to use an overlay which
references external files, and have them available for the
main (overlayed) definition.

Thanks @JakeSCahill for noticing the bug!
@paulRbr paulRbr force-pushed the dont-resolve-refs-on-overlay-files branch from 28b984b to 339346e Compare July 21, 2025 12:14
@paulRbr paulRbr enabled auto-merge July 21, 2025 12:14
@paulRbr paulRbr merged commit 2180b90 into bump-sh:main Jul 21, 2025
9 checks passed
@paulRbr paulRbr deleted the dont-resolve-refs-on-overlay-files branch July 21, 2025 12:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants