Skip to content

Re-enable the Pipenv Python version mismatch test - #2156

Merged
edmorley merged 1 commit into
mainfrom
edmorley/enable-pipenv-mismatched-python-version-test
Oct 9, 2026
Merged

edmorley merged 1 commit into
mainfrom
edmorley/enable-pipenv-mismatched-python-version-test

Conversation

@edmorley

@edmorley edmorley commented Oct 9, 2026

Copy link
Copy Markdown
Member

The upstream Pipenv bug that caused this test to be skipped has been fixed, so the test no longer needs to be skipped:
pypa/pipenv#6514

Pipenv now prints a DeployException traceback when aborting the deploy rather than its usage message, so the expected output has been updated to match.

Also removes the "Pipfile.lock python_version and .python-version" test and its fixture, since the re-enabled test covers the same case.

GUS-W-24469687.

@edmorley edmorley self-assigned this Oct 9, 2026
The upstream Pipenv bug that caused this test to be skipped has been
fixed, so the test no longer needs to be skipped:
pypa/pipenv#6514

Pipenv now prints a `DeployException` traceback when aborting the
deploy rather than its usage message, so the expected output has been
updated to match.

Also removes the "Pipfile.lock `python_version` and `.python-version`"
test and its fixture, since the re-enabled test covers the same case.

GUS-W-24469687.
@edmorley
edmorley force-pushed the edmorley/enable-pipenv-mismatched-python-version-test branch from 0abd9e0 to 7394b45 Compare October 9, 2026 13:32
@edmorley
edmorley marked this pull request as ready for review October 9, 2026 13:36
@edmorley
edmorley requested a review from a team as a code owner October 9, 2026 13:36
@edmorley
edmorley enabled auto-merge (squash) October 9, 2026 13:36
@edmorley
edmorley merged commit 4e70d6f into main Oct 9, 2026
10 checks passed
@edmorley
edmorley deleted the edmorley/enable-pipenv-mismatched-python-version-test branch October 9, 2026 14:17
@heroku-linguist heroku-linguist Bot mentioned this pull request Oct 9, 2026
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.

2 participants