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

Lock pip version in tests #2929

Closed
wants to merge 1 commit into from

Conversation

jxltom
Copy link
Contributor

@jxltom jxltom commented Oct 6, 2018

The issue

To have a stable and repeatable tests, the pip version in tests should be pinned. Otherwise, unexpected bugs such as #2924 may fail all CI tests. Passed tests may fail even codebase is not changed as time varies.

The pip version in tests should be the latest but a specific version. After #2924 is fixed, it should be pinned back to 18.1 again.

How does that sound?

The fix

Pin pip version in tests to latest but a specific version.

The checklist

  • Associated issue
  • A news fragment in the news/ directory to describe this fix with the extension .bugfix, .feature, .behavior, .doc. .vendor. or .trivial (this will appear in the release changelog). Use semantic line breaks and name the file after the issue number or the PR #.

@jxltom
Copy link
Contributor Author

jxltom commented Oct 6, 2018

Woops, it looks like the CI still has two failed tests even system pip version is not 18.1.

No idea on why pipenv graph still fails even system pip is downgrades. It may relate to sarugaku/pip-shims#17.

@techalchemy
Copy link
Member

The goal is actually that pipenv can build against any reasonable version of pip, so CI failures are highly desirable for catching this

@techalchemy
Copy link
Member

(i.e. if you as a user have anything between pip 9.x and 18.x, pipenv should still work)

@jxltom
Copy link
Contributor Author

jxltom commented Oct 7, 2018

Yeah that will be perfect.

But sometime unexpected bug may block all commits or PR being merged, since they are not actually passed the tests.

But this PR doesn't fix the test (no idea why). So I think it is not helping anyway.

@techalchemy
Copy link
Member

(don't spend too much time looking, i'm just looking for issue #s before i push mine)

@jxltom
Copy link
Contributor Author

jxltom commented Oct 7, 2018

Great, looking forward to the new release!

@jxltom
Copy link
Contributor Author

jxltom commented Oct 8, 2018

Close becasue #2935

@jxltom jxltom closed this Oct 8, 2018
@jxltom jxltom deleted the lock-pip-version-in-tests branch October 8, 2018 05:17
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