Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jinja templating for config file formats #700

Closed
danielabdi-noaa opened this issue Mar 27, 2023 · 2 comments
Closed

Jinja templating for config file formats #700

danielabdi-noaa opened this issue Mar 27, 2023 · 2 comments
Assignees
Labels
enhancement New feature or request

Comments

@danielabdi-noaa
Copy link
Collaborator

danielabdi-noaa commented Mar 27, 2023

Description

Currently config files support jinja templating using extend_yaml function that fills values of templated jinja2 entries refering to variables in the same config file. However, support for jinja templating of a block of yaml entries is not supported. This may come in handy, for example, when you want to define a set of variables only if a certain condition is met. With the existing approach, one has to define the variables and set to null/empty even when they are not needed.

Solution

Add jinja templating to all config file formats: yaml, json, xml, ini, and shell

Requirements**

None

Acceptance Criteria (Definition of Done)

Fundamental tests pass

@gspetro-NOAA
Copy link
Collaborator

@danielabdi-noaa @christinaholtNOAA Since the related PR was closed, can this issue be closed?

@christinaholtNOAA
Copy link
Collaborator

This will not be included.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants