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

Isolating pulser-pasqal version and release process #733

Merged
merged 3 commits into from
Sep 24, 2024

Conversation

HGSilveri
Copy link
Collaborator

@HGSilveri HGSilveri commented Sep 23, 2024

  • Creates its own VERSION.txt file for pulser-pasqal
  • Removes pulser-pasqal from the list of pinned packages in the pulser metapackage (ie packages.txt) -> pulser-pasqal will no longer be released alongside pulser-core and pulser-simulation
  • Makes pulser-pasqal an unpinned dependency of pulser (ie it will be up to pulser-pasqal to dictate which versions of pulser-core it is compatible with)
  • Relaxes pulser-pasqal's dependency on pulser-core to >=0.20
  • Bumps pulser-pasqal to 0.20.1 -> Merging these changes should be immediately followed by a manual release of pulser-pasqal on its own

@HGSilveri HGSilveri self-assigned this Sep 23, 2024
@HGSilveri HGSilveri added this to the Pulser v1 milestone Sep 23, 2024
Copy link
Collaborator

@a-corni a-corni left a comment

Choose a reason for hiding this comment

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

LGTM :)

Copy link
Collaborator

@oliver-gordon oliver-gordon left a comment

Choose a reason for hiding this comment

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

If this is it finally being unpinned, we can certainly start work next sprint on moving to a new repo

@HGSilveri HGSilveri merged commit cea1f7a into develop Sep 24, 2024
9 checks passed
@HGSilveri HGSilveri deleted the hs/isolate-pulser-pasqal branch November 12, 2024 11:18
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.

4 participants