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

Bump pip-tools version #1215

Merged
merged 6 commits into from
Feb 7, 2022
Merged

Bump pip-tools version #1215

merged 6 commits into from
Feb 7, 2022

Conversation

antonymilne
Copy link
Contributor

@antonymilne antonymilne commented Feb 4, 2022

Description

Fixes #1208. In short, pip-tools==6.4.0 is not compatible with the recently released pip>=22 . pip-tools have now released a fix version 6.5.0 which does work with pip>=22, but it's only compatible with Python 3.7+.

I've also reversed the changes to CI made in #1189 so that the pip requirement is now open-ended again. I actually think this is a good system because if pip releases a new version that breaks our CI it gives us an immediate warning that other stuff will likely break also.

Checklist

  • Read the contributing guidelines
  • Opened this PR as a 'Draft Pull Request' if it is work-in-progress
  • Updated the documentation to reflect the code changes
  • Added a description of this change in the RELEASE.md file
  • Added tests to cover my changes

Signed-off-by: Antony Milne <antony.milne@quantumblack.com>
Signed-off-by: Antony Milne <antony.milne@quantumblack.com>
Signed-off-by: Antony Milne <antony.milne@quantumblack.com>
Signed-off-by: Antony Milne <antony.milne@quantumblack.com>
@antonymilne antonymilne self-assigned this Feb 4, 2022
@antonymilne antonymilne marked this pull request as ready for review February 4, 2022 13:29
RELEASE.md Outdated
@@ -9,6 +9,7 @@
* Added Plotly dataset documentation (`03_tutorial/06_visualise_pipeline.md`).
* Added the upper limit `pandas<1.4` to maintain compatibility with `xlrd~=1.0`.
* Bumped the `Pillow` minimum version requirement to 9.0 (Python 3.7+ only) following [CVE-2022-22817](https://cve.mitre.org/cgi-bin/cvename.cgi?name=CVE-2022-22817).
* Upgraded `pip-tools`, which is used by `kedro build-reqs`, to 6.5 (Python 3.7+ only). This `pip-tools` version is compatible with `pip>=21.2`, including the most recent releases of `pip`.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we make a mention that for users still on python 3.6, they need to use older version of pip, i.e. <22? Or did I misunderstand this?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Python 3.6 users will be on pip<22 anyway, but I've added a note to be explicit here (since I also had to think about this for a while to figure out what was going on...😀 )

Makefile Show resolved Hide resolved
RELEASE.md Outdated Show resolved Hide resolved
@antonymilne antonymilne merged commit 07a14fe into main Feb 7, 2022
@antonymilne antonymilne deleted the chore/bump-pip-tools branch February 7, 2022 17:14
Galileo-Galilei pushed a commit to Galileo-Galilei/kedro that referenced this pull request Feb 19, 2022
lvijnck pushed a commit to lvijnck/kedro that referenced this pull request Apr 7, 2022
Signed-off-by: Laurens Vijnck <laurens_vijnck@mckinsey.com>
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.

Kedro build-reqs breaks with pip>=22.0
3 participants