Skip to content

Conversation

@dabacon
Copy link
Collaborator

@dabacon dabacon commented May 19, 2022

  • also make a constant ALL_CAPS and use simpler type annotation

#5353

@dabacon dabacon requested review from a team, cduck and vtomole as code owners May 19, 2022 23:13
@dabacon dabacon requested a review from 95-martin-orion May 19, 2022 23:13
@CirqBot CirqBot added the size: S 10< lines changed <50 label May 19, 2022
@dabacon dabacon requested a review from mpharrigan May 19, 2022 23:13
return base
return f'({base}**{proper_repr(self._exponent)})'

def _json_dict_(self):
Copy link
Collaborator

Choose a reason for hiding this comment

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

Nit: Add return type

@dabacon
Copy link
Collaborator Author

dabacon commented May 19, 2022

Strange lint error

@dabacon dabacon added the automerge Tells CirqBot to sync and merge this PR. (If it's running.) label May 20, 2022
@CirqBot CirqBot added the front_of_queue_automerge CirqBot uses this label to indicate (and remember) what's being merged next. label May 20, 2022
@CirqBot CirqBot merged commit 70fdf1f into quantumlib:master May 20, 2022
@CirqBot CirqBot removed automerge Tells CirqBot to sync and merge this PR. (If it's running.) front_of_queue_automerge CirqBot uses this label to indicate (and remember) what's being merged next. labels May 20, 2022
rht pushed a commit to rht/Cirq that referenced this pull request May 1, 2023
* also make a constant ALL_CAPS and use simpler type annotation

quantumlib#5353
harry-phasecraft pushed a commit to PhaseCraft/Cirq that referenced this pull request Oct 31, 2024
* also make a constant ALL_CAPS and use simpler type annotation

quantumlib#5353
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.

3 participants