Skip to content

Add include resolver and remove automatic inclusion of files with specific file extensions. - #465

Merged
gubaidulinvadim merged 3 commits into
mainfrom
include-resolver
Oct 6, 2026
Merged

gubaidulinvadim merged 3 commits into
mainfrom
include-resolver

Conversation

@TeresiaOlsson

@TeresiaOlsson TeresiaOlsson commented Oct 5, 2026 •

Copy link
Copy Markdown
Member

I have made the changes to switch the behaviour of the resolvers for including files discussed in #437.

This is what I did:

  • Renamed the file resolver to include.
  • Removed that strings with some specific suffices would automatically be interpreted as files to load and expand. Now you need to explicitly write ${include: ...} for it to be loaded and expanded.
  • Updated the tests accordingly. That includes updating all the config files used for the tests since they make heavy use of this.
  • Keep the path resolver for now since I know there are things in the documentation which uses it. It should be possible to remove it after the documentation tutorials are updated.

Only part I found maybe not so nice is that you now need to use ${include: ...} also for the catalog but I think that only affects the static catalog?

@gubaidulinvadim
gubaidulinvadim merged commit 12a9153 into main Oct 6, 2026
4 checks passed
@gubaidulinvadim
gubaidulinvadim deleted the include-resolver branch October 6, 2026 08:03
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