-
-
Notifications
You must be signed in to change notification settings - Fork 524
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
passenv
regression when multiple variables are named on the same line
#2615
Comments
I'll need to see if we can support this, or we'll mark this as a known breaking change 🤔 in the meantime can spread it over multiple lines as a workaround. If you have time PR is welcome. |
Please make it work with spaces :) otherwise hundreds of projects will have to change, and some of mine will need to list a dozen lines... would be awesome if this was not a breaking change |
Well, asking is easy, making it much harder and might have other ramifications in other places 🤷 sensible PRs welcome. |
For what is worth while separating the variables with space does not work, you can still put these in one line by using instead the [testenv:inline]
passenv = FOO, BAR instead of [testenv:inline]
passenv = FOO BAR Took a look at the codebase, and there's no easy way to support space without significant problems in other areas, so we'll just have to document this as breaking change with the here described workaround (use |
May I suggest to add this to the upgrade FAQ? |
That is an excellent suggestion. I will create a PR once I have a chance. |
tox4 no longer allows space-separated `passenv` tox-dev/tox#2615 however tox3 doesn't support `VAR, ` either so to support both versions of tox, use this funky space + comma separated syntax, which really does work.
Due to a regression in tox, passenv now needs a comma separated list. This is necessary for our CI to upload to PyPI automagically. See tox-dev/tox#2615 for more info.
tox4 no longer allows space-separated `passenv`: tox-dev/tox#2615 however tox3 doesn't support `VAR, ` so to support both versions of tox, use this funky space + comma separated syntax, which really does work.
Issue
In tox 4, space-separated variables specified for passenv are not passed into the container.
☝️ doesn't work on tox 4
☝️ works fine on tox3 and tox4
Environment
Provide at least:
pip list
of the host Python wheretox
is installed:Output of running tox
Provide the output of
tox -rvv
:Minimal example
The text was updated successfully, but these errors were encountered: