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

tests: test against multiple Qiskit versions #143

Draft
wants to merge 26 commits into
base: master
Choose a base branch
from

Conversation

airwoodix
Copy link
Collaborator

@airwoodix airwoodix commented Feb 23, 2024

Summary

Improve the testing coverage:

  • test examples against the installed packages instead of the source checkout
  • test against multiple (hand-selected) versions of Qiskit.

Since the package is still compatible with Qiskit 0.46, lower the bound accordingly (was raised in #141).

Details and comments

The build system is migrated from Poetry to Hatch / Hatchlings and the dependency resolution on CI to uv. The latter provides dependency resolution overrides, which are used for the dependency matrix testing. However, it is still quite experimental, thus the number of workarounds.

Windows tests (examples on the built package only, as previously) shall be re-enabled later, as they require a platform-agnostic runner for the examples.

@airwoodix airwoodix self-assigned this Feb 23, 2024
@airwoodix airwoodix force-pushed the project-uv-hatch branch 3 times, most recently from ccd0af6 to f5621bd Compare February 23, 2024 13:44
@wilfried-huss
Copy link
Collaborator

Hatch has now native support for UV.

Maybe this can help to simplify this pull request.

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