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

Allow configuring parameters for Nunjucks #278

Open
dhruvkb opened this issue Nov 22, 2022 · 0 comments
Open

Allow configuring parameters for Nunjucks #278

dhruvkb opened this issue Nov 22, 2022 · 0 comments
Labels
bug Something isn't working good first issue Good for newcomers help wanted Extra attention is needed

Comments

@dhruvkb
Copy link
Contributor

dhruvkb commented Nov 22, 2022

🐞 Describe the bug

Currently some configuration options for Nunjucks are hardcoded into the action.

nunjucks.configure({ autoescape: true, trimBlocks: true, lstripBlocks: true })

These can be incompatible with the needs of a project. For example, in a YAML project, autoescape can cause single quotes (') to be replaced with HTML entity codes (').

📚 To reproduce

Use a template-based approach to write YAML containing single quotes.

💡 Expected behavior

Nunjucks should be configurable by setting some options on the action that are passed into the nunjucks.configure function.

🖼️ Screenshots

No response

⚙️ Environment

v1.18.0

📋 Additional context

I'd be happy to open a PR to resolve this.

@dhruvkb dhruvkb added the bug Something isn't working label Nov 22, 2022
@BetaHuhn BetaHuhn added help wanted Extra attention is needed good first issue Good for newcomers labels Nov 30, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working good first issue Good for newcomers help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

2 participants