Skip to content

Conversation

@DaRasch
Copy link
Contributor

@DaRasch DaRasch commented Oct 15, 2025

Implements the feature requested in #1485

It is now possible to override the default template loader used in the files.template operation by passing a custom instance as shown in the example in the issue.

I did not add any new tests, as I did not understand how the test system works. I also did not find any tests for the function I changed 🤷

  • Pull request is based on the default branch (3.x at this time)
  • Pull request includes tests for any new/updated operations/facts
  • Pull request includes documentation for any new/updated operations/facts
  • Tests pass (see scripts/dev-test.sh)
  • Type checking & code style passes (see scripts/dev-lint.sh)

Copy link
Member

@Fizzadar Fizzadar left a comment

Choose a reason for hiding this comment

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

Very nice, thank you @DaRasch!

Testing this function via the usual JSON/YAML tests is non-trivial, so OK to skip for now.

@Fizzadar Fizzadar merged commit 52cfa3a into pyinfra-dev:3.x Oct 21, 2025
25 checks passed
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