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 Python version in CI #905

Merged
merged 4 commits into from
Jun 20, 2023
Merged

Conversation

mtreinish
Copy link
Member

Recently github updated the cached version of python installed for 3.7 in the CI environment to 3.7.17. This new binary was not built with bz2 support which is causing failures in our CI jobs that run with 3.7. While we'll be dropping 3.7 support from the main branch (for 0.14.0) in the near future we still support 3.7 on the stable 0.13.x series. This commit pins the python version to the previous patch release which was known to work.

Recently github updated the cached version of python installed for 3.7
in the CI environment to 3.7.17. This new binary was not built with bz2
support which is causing failures in our CI jobs that run with 3.7.
While we'll be dropping 3.7 support from the main branch (for 0.14.0) in
the near future we still support 3.7 on the stable 0.13.x series. This
commit pins the python version to the previous patch release which was
known to work.
@mtreinish mtreinish added the stable-backport-potential This PR or issue is potentially worth backporting for inclusion in a stable branch label Jun 19, 2023
@IvanIsCoding IvanIsCoding added the automerge Queue a approved PR for merging label Jun 19, 2023
@IvanIsCoding
Copy link
Collaborator

LGTM, we should probably also file a bug somewhere for the maintainers of the action to track it

@coveralls
Copy link

Pull Request Test Coverage Report for Build 5314435053

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • 3 unchanged lines in 1 file lost coverage.
  • Overall coverage increased (+0.01%) to 96.467%

Files with Coverage Reduction New Missed Lines %
rustworkx-core/src/connectivity/all_simple_paths.rs 3 97.44%
Totals Coverage Status
Change from base Build 5294336008: 0.01%
Covered Lines: 15043
Relevant Lines: 15594

💛 - Coveralls

@IvanIsCoding IvanIsCoding merged commit 5f73e76 into Qiskit:main Jun 20, 2023
29 checks passed
mergify bot pushed a commit that referenced this pull request Jun 20, 2023
* Pin Python version in CI

Recently github updated the cached version of python installed for 3.7
in the CI environment to 3.7.17. This new binary was not built with bz2
support which is causing failures in our CI jobs that run with 3.7.
While we'll be dropping 3.7 support from the main branch (for 0.14.0) in
the near future we still support 3.7 on the stable 0.13.x series. This
commit pins the python version to the previous patch release which was
known to work.

* Use 3.7 on windows

* Fix windows capitalization

* Fix syntax error

(cherry picked from commit 5f73e76)
mtreinish added a commit that referenced this pull request Jun 20, 2023
* Pin Python version in CI

Recently github updated the cached version of python installed for 3.7
in the CI environment to 3.7.17. This new binary was not built with bz2
support which is causing failures in our CI jobs that run with 3.7.
While we'll be dropping 3.7 support from the main branch (for 0.14.0) in
the near future we still support 3.7 on the stable 0.13.x series. This
commit pins the python version to the previous patch release which was
known to work.

* Use 3.7 on windows

* Fix windows capitalization

* Fix syntax error

(cherry picked from commit 5f73e76)

Co-authored-by: Matthew Treinish <mtreinish@kortar.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge Queue a approved PR for merging stable-backport-potential This PR or issue is potentially worth backporting for inclusion in a stable branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants