Skip to content

Fix docstring indentation of cirq.PauliString - #5940

Merged
tanujkhattar merged 2 commits into
quantumlib:masterfrom
tanujkhattar:pauli_string_docstring
Nov 3, 2022
Merged

tanujkhattar merged 2 commits into
quantumlib:masterfrom
tanujkhattar:pauli_string_docstring

Conversation

@tanujkhattar

Copy link
Copy Markdown
Collaborator

Code block of cirq.PauliString is rendered incorrectly due to indentation. See https://quantumai.google/reference/python/cirq/PauliString#used-in-the-notebooks

This PR changes the indentation to match the previous code block that gets rendered correctly.

@tanujkhattar
tanujkhattar requested review from a team, cduck and vtomole as code owners November 3, 2022 20:43
@tanujkhattar
tanujkhattar requested a review from viathor November 3, 2022 20:43
@CirqBot CirqBot added the size: S 10< lines changed <50 label Nov 3, 2022
@tanujkhattar tanujkhattar changed the title Fix docstring indentation cirq.PauliString Fix docstring indentation of cirq.PauliString Nov 3, 2022
@pavoljuhas

pavoljuhas commented Nov 3, 2022 •

Copy link
Copy Markdown
Collaborator

@tanujkhattar

Copy link
Copy Markdown
Collaborator Author

@pavoljuhas Updated. PTAL.

@pavoljuhas pavoljuhas left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

@tanujkhattar
tanujkhattar merged commit c485504 into quantumlib:master Nov 3, 2022
rht pushed a commit to rht/Cirq that referenced this pull request May 1, 2023
* Fix docstring indentation cirq.PauliString

* Update indentation for GridQubit, GridQid and flatten
harry-phasecraft pushed a commit to PhaseCraft/Cirq that referenced this pull request Oct 31, 2024
* Fix docstring indentation cirq.PauliString

* Update indentation for GridQubit, GridQid and flatten
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size: S 10< lines changed <50

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants