Skip to content

Fix passing multiple values in testenv.passenv in tox.ini#399

Merged
srprash merged 2 commits intoaws:masterfrom
musicinmybrain:tox-pass-env
Oct 16, 2023
Merged

Fix passing multiple values in testenv.passenv in tox.ini#399
srprash merged 2 commits intoaws:masterfrom
musicinmybrain:tox-pass-env

Conversation

@musicinmybrain
Copy link
Copy Markdown
Contributor

Current versions of tox advise: pass_env values cannot contain whitespace, use comma to have multiple values in a single line

Issue #, if available:
Fixes #398.

Description of changes:

Separates testenv.passenv entries with commas rather than spaces in tox.ini.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@musicinmybrain musicinmybrain requested a review from a team as a code owner August 7, 2023 14:44
Current versions of tox advise: pass_env values cannot contain
whitespace, use comma to have multiple values in a single line

Fixes aws#398.
@srprash
Copy link
Copy Markdown
Contributor

srprash commented Oct 16, 2023

Failure is unrelated to this change. Approving.
Thanks @musicinmybrain for contributing.

@srprash srprash merged commit 9d27fe2 into aws:master Oct 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Current tox versions do not like how testenv.passenv is set

2 participants