Skip to content

Upgrade PyO3 to 0.20.0.#11117

Merged
jakelishman merged 1 commit into
Qiskit:mainfrom
kevinhartman:pyo3-0.20.0
Oct 27, 2023
Merged

Upgrade PyO3 to 0.20.0.#11117
jakelishman merged 1 commit into
Qiskit:mainfrom
kevinhartman:pyo3-0.20.0

Conversation

@kevinhartman
Copy link
Copy Markdown
Contributor

Summary

Version bump of Rust dependency pyo3 to 0.20.0.

Details and comments

PyO3's changelog: https://github.com/PyO3/pyo3/blob/main/CHANGELOG.md#packaging

@kevinhartman kevinhartman added Changelog: None Do not include in the GitHub Release changelog. Rust This PR or issue is related to Rust code in the repository dependencies Pull requests that update a dependency file labels Oct 26, 2023
@kevinhartman kevinhartman requested a review from a team as a code owner October 26, 2023 14:04
@qiskit-bot
Copy link
Copy Markdown
Collaborator

One or more of the the following people are requested to review this:

Copy link
Copy Markdown
Member

@jakelishman jakelishman left a comment

Choose a reason for hiding this comment

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

This looks sensible, and I think our compile times will appreciate having only one version of syn in our lock. Which particular features from 0.20 are you after?

@jakelishman jakelishman enabled auto-merge October 26, 2023 14:14
@jakelishman
Copy link
Copy Markdown
Member

The build failure is probably unrelated to the PyO3 bump: Matt hit the same failure path in #11115.

@kevinhartman
Copy link
Copy Markdown
Contributor Author

Which particular features from 0.20 are you after?

@jakelishman I'm keen to start using the __eq__ and __ne__ methods, so we can avoid implementing __richcmp__ for the new CircuitData stuff 😄.

@jakelishman jakelishman added this pull request to the merge queue Oct 26, 2023
Merged via the queue into Qiskit:main with commit 57d8c64 Oct 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Changelog: None Do not include in the GitHub Release changelog. dependencies Pull requests that update a dependency file Rust This PR or issue is related to Rust code in the repository

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants