Skip to content

Add support for Jinja environment options in schemachange configuration.#227

Open
rosspb3 wants to merge 1 commit intoSnowflake-Labs:masterfrom
rosspb3:Jinja_config
Open

Add support for Jinja environment options in schemachange configuration.#227
rosspb3 wants to merge 1 commit intoSnowflake-Labs:masterfrom
rosspb3:Jinja_config

Conversation

@rosspb3
Copy link
Copy Markdown
Contributor

@rosspb3 rosspb3 commented Feb 7, 2024

Added ability to configure some Jinja Environment settings within the schemachange configuration via a new "jinja" section. This section would allow users to customize several parsing and rendering options in Jinja like trim_blocks lstrip_blocks, block_start_string, block_end_string, variable_start_string, variable_end_string, but disallows changing settings which could impact the behavior of schemachange like undefined, loader, enable_async.

@sfc-gh-tmathew sfc-gh-tmathew added enhancement New feature or request Under Review This is being discussed without planned changes labels Mar 6, 2024
@sfc-gh-tmathew sfc-gh-tmathew added the community-contribution Submitted by community label Nov 18, 2025
@sfc-gh-tmathew
Copy link
Copy Markdown
Collaborator

sfc-gh-tmathew commented Feb 9, 2026

Hi @rosspb3,

Thank you for this PR to add Jinja environment configuration options (trim_blocks, lstrip_blocks, etc.).

This PR has been open for about 2 years and likely has significant conflicts with the current codebase. We've also had other Jinja-related changes since then (including PR #353 for data injection).

Questions:

  1. Are you still interested in pursuing this feature?
  2. If so, would you be able to rebase against the current master branch?

If we don't hear back in the next few weeks, we may close this PR. You're welcome to reopen or create a new PR if you'd like to continue this work later.

Thanks for your contribution!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

community-contribution Submitted by community enhancement New feature or request Under Review This is being discussed without planned changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants