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

Pin pip for "pip check" #630

Merged
merged 1 commit into from
Aug 5, 2024
Merged

Conversation

t-imamichi
Copy link
Collaborator

@t-imamichi t-imamichi commented Aug 5, 2024

Summary

pip check of CI fails since a couple of days ago.

gurobipy 11.0.3 is not supported on this platform

I found an issue of pip pypa/pip#12884 is likely to be related to the failure. I try to pin pip only for pip check.

Details and comments

@t-imamichi t-imamichi added the type: ci Related to CI changes label Aug 5, 2024
@coveralls
Copy link

Pull Request Test Coverage Report for Build 10242188201

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 92.857%

Totals Coverage Status
Change from base Build 10042117970: 0.0%
Covered Lines: 4446
Relevant Lines: 4788

💛 - Coveralls

@t-imamichi t-imamichi added the stable backport potential The bug might be minimal and/or import enough to be port to stable label Aug 5, 2024
@t-imamichi t-imamichi mentioned this pull request Aug 5, 2024
Copy link
Member

@woodsp-ibm woodsp-ibm left a comment

Choose a reason for hiding this comment

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

LGTM - thx for linking the pip issue, was helpful to understand

@woodsp-ibm woodsp-ibm merged commit e4affd4 into qiskit-community:main Aug 5, 2024
16 checks passed
mergify bot pushed a commit that referenced this pull request Aug 5, 2024
(cherry picked from commit e4affd4)
@t-imamichi t-imamichi deleted the pin-pip branch August 6, 2024 03:04
mergify bot added a commit that referenced this pull request Aug 6, 2024
(cherry picked from commit e4affd4)

Co-authored-by: Takashi Imamichi <[email protected]>
@mergify mergify bot mentioned this pull request Aug 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
stable backport potential The bug might be minimal and/or import enough to be port to stable type: ci Related to CI changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants