a8m / a8m/envsubst

Add option to only parse $ALL_CAPS env vars

Open
#32 2 comments 0 reactions 0 assignees View on GitHub
Dominant language
Go
Stars
907
Forks
96
PR merge metrics
No merged PRs in 30d

Description

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.

Contributor guide

No contributing guide indexed for this repository

Research direction

Look at the parsing logic in the main package to understand how environment variables are currently detected. The change involves adding a flag to filter matches to only uppercase variables like $ALL_CAPS. Check existing tests for substitution behavior to add a test for the new flag. Running the CLI with sample nginx configs will verify the flag works as intended.

Written by the indexing model from the issue text.

Assessment

Tech stack
go
Domain
cli
Issue type
Feature
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Clearly specified
Newbie friendliness
65/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.