Skip to content

Fix CI actions on GitHub#672

Merged
pavoljuhas merged 33 commits into
quantumlib:masterfrom
pavoljuhas:fix-ci-actions
Sep 14, 2024
Merged

Fix CI actions on GitHub#672
pavoljuhas merged 33 commits into
quantumlib:masterfrom
pavoljuhas:fix-ci-actions

Conversation

@pavoljuhas
Copy link
Copy Markdown
Collaborator

@pavoljuhas pavoljuhas commented Sep 4, 2024

  • update linux runners to ubuntu-22 or ubuntu-latest, but run test-mem
    job on ubuntu-20 for it for some reason fails on ubuntu-22
  • bump up CI actions to be compatible with node 20, namely bump up to
    actions/checkout@v4, actions/setup-python@v5, download-artifact@v4, upload-artifact@v4
  • update steps involving download-artifact and upload-artifact actions per
    https://github.com/actions/download-artifact/blob/main/docs/MIGRATION.md
  • migrate to docker compose v2
  • to build binary wheels for mac os, set MACOSX_DEPLOYMENT_TARGET according
    to the macos version
  • stop building and testing wheels for old Python versions 3.7, 3.8

pavoljuhas and others added 28 commits September 4, 2024 13:21
Recent GHA runner images require docker-compose v2.

Ref: actions/runner-images#9557
That version is possibly too old.  Wheels for newer Python versions
appear to build successfully.
It is nearing its end-of-life.
Copy link
Copy Markdown
Collaborator

@95-martin-orion 95-martin-orion left a comment

Choose a reason for hiding this comment

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

All checks passing - LGTM. Thanks for digging into this!

There are a few in-flight PRs that should be revisited now that they're unblocked by this.

Per GHA annotations all brew libraries are already installed.
@pavoljuhas pavoljuhas marked this pull request as ready for review September 14, 2024 22:29
@pavoljuhas pavoljuhas merged commit cd646a9 into quantumlib:master Sep 14, 2024
@pavoljuhas pavoljuhas deleted the fix-ci-actions branch September 14, 2024 23:05
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