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

Add missing expr.bit_xor documentation #10828

Merged
merged 1 commit into from
Sep 13, 2023

Conversation

jakelishman
Copy link
Member

Summary

Apparently I forgot to build this one function into the documentation, even though its docstring was already written.

Details and comments

@jakelishman jakelishman added documentation Something is not clear or an error documentation stable backport potential The bug might be minimal and/or import enough to be port to stable Changelog: None Do not include in changelog labels Sep 13, 2023
@jakelishman jakelishman requested a review from a team as a code owner September 13, 2023 10:18
@qiskit-bot
Copy link
Collaborator

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

  • @Qiskit/terra-core

@mtreinish mtreinish enabled auto-merge September 13, 2023 10:31
@coveralls
Copy link

Pull Request Test Coverage Report for Build 6171127023

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • 11 unchanged lines in 3 files lost coverage.
  • Overall coverage decreased (-0.01%) to 87.267%

Files with Coverage Reduction New Missed Lines %
crates/qasm2/src/expr.rs 1 93.76%
crates/qasm2/src/lex.rs 4 90.91%
crates/qasm2/src/parse.rs 6 97.6%
Totals Coverage Status
Change from base Build 6166101121: -0.01%
Covered Lines: 74193
Relevant Lines: 85018

💛 - Coveralls

@mtreinish mtreinish added this pull request to the merge queue Sep 13, 2023
Merged via the queue into Qiskit:main with commit 579084e Sep 13, 2023
13 checks passed
mergify bot pushed a commit that referenced this pull request Sep 13, 2023
@jakelishman jakelishman deleted the doc-expr-bit-xor branch September 13, 2023 21:36
github-merge-queue bot pushed a commit that referenced this pull request Sep 14, 2023
(cherry picked from commit 579084e)

Co-authored-by: Jake Lishman <[email protected]>
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 changelog documentation Something is not clear or an error documentation stable backport potential The bug might be minimal and/or import enough to be port to stable
Projects
No open projects
Status: Done
Development

Successfully merging this pull request may close these issues.

4 participants