Fix issue #16097 - Fix qasm3.dumps_experimental delay units#16114
Fix issue #16097 - Fix qasm3.dumps_experimental delay units#16114pedro2mj wants to merge 6 commits intoQiskit:mainfrom
qasm3.dumps_experimental delay units#16114Conversation
|
Thank you for opening a new pull request. Before your PR can be merged it will first need to pass continuous integration tests and be reviewed. Sometimes the review process can be slow, so please be patient. While you're waiting, please feel free to review other open PRs. While only a subset of people are authorized to approve pull requests for merging, everyone is encouraged to review open pull requests. Doing reviews helps reduce the burden on the core team and helps make the project's code better for everyone. One or more of the following people are relevant to this code:
|
mtreinish
left a comment
There was a problem hiding this comment.
This change looks correct to me, thanks for fixing it. But before we merge it could you add a test that covers these fixes. Additionally, could you add a release note for the fix. The process for this is documented here: https://github.com/Qiskit/qiskit/blob/main/CONTRIBUTING.md#release-notes
Coverage Report for CI Build 25164307073Warning Build has drifted: This PR's base is out of sync with its target branch, so coverage data may include unrelated changes. Coverage increased (+0.02%) to 87.583%Details
Uncovered Changes
Coverage Regressions7 previously-covered lines in 3 files lost coverage.
Coverage Stats
💛 - Coveralls |
qasm3.dumps_experimental delay units
AI/LLM disclosure