Qiskit / qiskit / 37513023385

88%

main: 88%

Build Type

Pull #17157

github

Committed by mergify[bot]

Commit Message

Ensure we check min_version constraint first in qpy.load() (#17152)

* Ensure we check min_version constraint first in qpy.load()

This commit fixes an oversight in the QPY load() function where the
min_version constraint wasn't checked before calling the rust version.
This meant if the min_version was set to a version >
QPY_RUST_READ_MIN_VERSION (which is currently 13) nothing would check
the configured min_version check. For the python interface fixing this
this is a simple matter of moving the version check prior to dispatching
to the rust enrtrypoint function. The C API also doesn't currently check
the config file. But that is a larger change and will be done in a
follow up.

* Update releasenotes/notes/fix-min-version-respect-gt-13-e1c986726ceb977d.yaml

Co-authored-by: Jake Lishman <jake@binhbar.com>

---------

Co-authored-by: Jake Lishman <jake@binhbar.com>
(cherry picked from commit a2f53b07d)

Pull Request Pull Request #17157: Ensure we check min_version constraint first in qpy.load() (backport #17152)

Coverage Stats

4 of 4 new or added lines in 1 file covered. (100.0%)

4 existing lines in 3 files now uncovered.

123444 of 140024 relevant lines covered (88.16%)

936271.56 hits per line