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

fix: upgrade pip-tools to fix bug in version 6.6.1 #1862

Merged

Conversation

MichaelRoytman
Copy link
Contributor

@MichaelRoytman MichaelRoytman commented Jun 8, 2022

TL;DR -

This commit upgrades the version of pip-tools used in this repository from 6.6.0 to 6.6.2.

In version 6.6.0 of pip-tools, there is a bug that is preventing pip-tools from working. This is breaking the Python requirements update GitHub action in this repository.

The error is ImportError: cannot import name 'BAR_TYPES' from 'pip._internal.cli.progress_bars'. The error was reported here. The fix to this bug was released in version 6.6.2 of pip-tools. See the comment here.

Version 6.6.1 of pip-tools also has a bug, which is fixed in version 6.6.2. I observed this issue breaking the Python requirements update GitHub action in another repository, so I have upgrade the version straight to 6.6.2. The issue in version 6.6.1 was reported here.

JIRA: None.

What changed?

See above.

Developer Checklist

Testing Instructions

  1. Run make upgrade.

Reviewer Checklist

Collectively, these should be completed by reviewers of this PR:

  • I've done a visual code review
  • I've tested the new functionality

FYI: @openedx/content-aurora

This commit upgrades the version of pip-tools used in this repository from 6.6.1 to 6.6.2.

In version 6.6.1 of pip-tools, there is a bug that is preventing pip-tools from working. This is breaking the Python requirements update GitHub action in this repository.

The error is "AttributeError: 'RequirementSet' object has no attribute 'add_requirement'". The error was reported here: jazzband/pip-tools#1623. The fix to this bug was released in version 6.6.2 of pip-tools. See the pull request here: jazzband/pip-tools#1624.
@MichaelRoytman MichaelRoytman requested a review from a team as a code owner June 8, 2022 18:23
@codecov
Copy link

codecov bot commented Jun 8, 2022

Codecov Report

Merging #1862 (281cf7a) into master (cffc145) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1862   +/-   ##
=======================================
  Coverage   94.95%   94.95%           
=======================================
  Files         153      153           
  Lines       17014    17014           
  Branches     1875     1875           
=======================================
  Hits        16155    16155           
  Misses        643      643           
  Partials      216      216           
Flag Coverage Δ
unittests 94.95% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.


Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update cffc145...281cf7a. Read the comment docs.

@MichaelRoytman MichaelRoytman merged commit 07b18d3 into master Jun 8, 2022
@MichaelRoytman MichaelRoytman deleted the mroytman/fix-pip-tools-bug-python-requirements branch June 8, 2022 18:57
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.

None yet

2 participants