Skip to content

Add relative content resolution - #71

Merged
ioquatix merged 2 commits into
mainfrom
relative-content-resolution
Aug 14, 2026
Merged

Add relative content resolution#71
ioquatix merged 2 commits into
mainfrom
relative-content-resolution

Conversation

@samuel-williams-shopify

Copy link
Copy Markdown
Contributor

Summary

  • add a <relative:...> namespace for resolving content from the logical node request path
  • preserve existing <content:...> resolution relative to the physical template location
  • fall back through parent directories when no nested relative override exists
  • document the new namespace and add nested override, root fallback, and physical-anchor coverage

Verification

  • focused relative content tests: 2 passed
  • RuboCop: 141 files inspected, no offenses
  • full test suite: 358 passed, 4 errored out of 362; the four command-generation tests failed because their subprocesses could not invoke a compatible bundle executable

@ioquatix
ioquatix merged commit e414923 into main Aug 14, 2026
34 of 40 checks passed
@ioquatix
ioquatix deleted the relative-content-resolution branch August 14, 2026 12:47
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