-
Notifications
You must be signed in to change notification settings - Fork 3
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
Support backticks inside defaults #10
Comments
#15 is a simpler case, worth doing before this one. |
The ability to inject sub_types into the tox config parser is a wonderful crazy idea which needs its own tox plugin helper library, with a dedicated regression test, or it needs support in tox. Without support inside tox, currently this would need to inject a modified |
FOO={env:FOO:blah}
is common enough that users would want to replaceblah
with backticks.i.e.
The problem with that is it takes away valid syntax, as previously that would result in FOO being equal to
do_stuff
in backticks. While that is not likely to be a common case, it was valid.Another option is to add a
sub_type
and thenFOO={env:FOO:{backticks:do_stuff}}
can be used. That should currently be an error "Unknown sub_type backticks", or similar, so it doesnt remove valid syntax.This can be done atm using
But then we run into #9 as DEFAULTFOO needs to be executed even if FOO was defined externally. But trying to determine that
DEFAULTFOO
is unneeded is hard, as the regex currently being used only returns the innermostenv:DEFAULTFOO
.The text was updated successfully, but these errors were encountered: