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.

Sign up for GitHub

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

Jump to bottom

Add option to only parse $ALL_CAPS env vars #32

Open
vladnosek opened this issue May 15, 2021 · 2 comments
Open

Add option to only parse $ALL_CAPS env vars #32

vladnosek opened this issue May 15, 2021 · 2 comments

Comments

@vladnosek
Copy link

Especially for nginx configs, lots of directives use $lowercase_variable, so having to escape a lot of content.

It would be nice to have an caps-only flag to only detect env vars that are uppercase.
@vladnosek vladnosek changed the title Add option to only parse $ALL_CAPS env vars Add option to only only parse $ALL_CAPS env vars May 15, 2021
@vladnosek vladnosek changed the title Add option to only only parse $ALL_CAPS env vars Add option to only parse $ALL_CAPS env vars May 15, 2021
@dnitsch
Copy link
Contributor

dnitsch commented Jun 11, 2021 •
edited
Loading

I could jump on this if there are no other takers for a PR on this.

@a8m the only thing I guess is to settle on the bool flag name --no-lower maybe?

@a8m
Copy link
Owner

a8m commented Jun 21, 2021

Hey @dnitsch and @vladnosek,
I'm fine with adding this option (including the new flag). Please feel free to send a PR and tag me there. Thanks 🙏

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@dnitsch @a8m @vladnosek