Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Fix section substitution with setenv #3289

Merged
merged 1 commit into from
Jun 5, 2024
Merged

Fix section substitution with setenv #3289

merged 1 commit into from
Jun 5, 2024

Conversation

JJLLWW
Copy link
Contributor

@JJLLWW JJLLWW commented Jun 5, 2024 •
edited
Loading

  • ran the linter to address style issues (tox -e fix)
  • wrote descriptive pull request text
  • ensured there are test(s) validating the fix
  • added news fragment in docs/changelog folder
  • updated/extended the documentation

Fixes #3262, resolving the environment variable name now causes the previous cache to be invalidated.

@JJLLWW JJLLWW requested a review from gaborbernat as a code owner June 5, 2024 11:27
@gaborbernat gaborbernat merged commit 489ad82 into tox-dev:main Jun 5, 2024
25 checks passed
seyidaniels pushed a commit to seyidaniels/tox that referenced this pull request Jun 10, 2024
@JJLLWW @seyidaniels
Fix section substitution with setenv (tox-dev#3289)
d8d1ff1
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@gaborbernat gaborbernat gaborbernat approved these changes

Assignees
No one assigned
Labels
bot:chronographer:provided
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Section substitution doesn't work in setenv
2 participants
@JJLLWW @gaborbernat