Fixed typo in dagcircuit.py#11077
Conversation
qiskit-bot
added
the
Community PR
|
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 the following people are requested to review this:
|
Pull Request Test Coverage Report for Build 6596228135
💛 - Coveralls |
mtreinish
left a comment
mtreinish
left a comment
There was a problem hiding this comment.
LGTM, thanks for catching and fixing these.
| ``False``, then the ``input_dag`` is assumed to faithfully implement suitable | ||
| conditional logic already. This is ignored for :class:`.ControlFlowOp`\\ s (i.e. | ||
| treated as if it is ``False``); replacements of those must already fulfil the same | ||
| treated as if it is ``False``); replacements of those must already fulfill the same |
There was a problem hiding this comment.
FWIW, I think the single "l" is more common in UK English, while two "l"s is used in US English. But, we use US English for Qiskit so it's fine to change this just in the interest of consistency even though both are valid.
4 participants
compatiblity -> compatibility
fulfil -> fulfill