Skip to content

Conversation

@edmorley
Copy link
Member

@edmorley edmorley commented Dec 8, 2025

In #1890 a warning was added if an existing Python virtual environment was found at .venv/ or venv/ in the root of the app dir, for the reasons described in that PR.

Now that 3 months have passed, this warning is being converted to an error.

Users with an inadvertently committed venv will need to follow the instructions in the error message to remove the venv directory from the Git index and add it to their .gitignore file.

GUS-W-19127259.

In #1890 a warning was added if an existing Python virtual environment
was found at `.venv/` or `venv/` in the root of the app dir, for the
reasons described in that PR.

Now that 3 months have passed, this warning is being converted to an
error.

Users with an inadvertently committed venv will need to follow the
instructions in the error message to remove the venv directory from
the Git index and add it to their `.gitignore` file.

GUS-W-19127259.
@edmorley edmorley self-assigned this Dec 8, 2025
@edmorley edmorley marked this pull request as ready for review December 8, 2025 15:22
@edmorley edmorley requested a review from a team as a code owner December 8, 2025 15:22
@edmorley edmorley merged commit 8997ea7 into main Dec 8, 2025
6 checks passed
@edmorley edmorley deleted the existing-venv-error branch December 8, 2025 18:00
@heroku-linguist heroku-linguist bot mentioned this pull request Dec 9, 2025
@edmorley
Copy link
Member Author

edmorley commented Dec 9, 2025

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.

2 participants