Skip to content
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

config: Revert removal of {packages}/{opts} hack #1801

Merged
merged 2 commits into from
Jan 12, 2021

Conversation

jayvdb
Copy link

@jayvdb jayvdb commented Jan 11, 2021

Closes #1777

Re-submitting #1794 which appears to be lost after merging.

Also I suspect the auto-closing didnt work, and I see now that is because the two commits with specific messages were merged into a single commit using the GitHub merge features.

This was removed in e4d0d60, and replaced with a
neater approach, however the newer approach doesnt
work for replacements from other sections.

Fixes tox-dev#1777
@jayvdb jayvdb changed the title test_config: Add test for indirect install_command config: Revert removal of {packages}/{opts} hack Jan 11, 2021
@gaborbernat gaborbernat merged commit ff4bd47 into tox-dev:master Jan 12, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Cannot substitute install_command from another section (regression in 3.21.0)
2 participants