Closed sylwang closed 1 year ago
…braces
Address 43818 so that the user can include variables with braces in the config file.
lex.go
${foo}
nginx.config
parse_test.go
lex_test.go
Before creating a PR, run through this checklist and mark each as complete.
CONTRIBUTING
README.md
Closing this pull request for now, and will reopen a PR for changes to the parsing logic after this PR for linter related changes gets approved.
…braces
Proposed changes
Address 43818 so that the user can include variables with braces in the config file.
lex.go
to correctly parse variables like${foo}
nginx.config
file in testdata for a test case of variables with bracesparse_test.go
for checking the parser can correctly parse variables with braceslex_test.go
since the values that are separated by spaces need to be tokenizedChecklist
Before creating a PR, run through this checklist and mark each as complete.
CONTRIBUTING
documentREADME.md
)