Skip to content

Conversation

@q-inho
Copy link

@q-inho q-inho commented Aug 11, 2025

Summary

Fix minor typos

  • Remove duplicate words "can"
  • CPHASE01 -> CPHASE10
  • test_run__no_measurments->test_run__no_measurements

Copy link

@windsurf-bot windsurf-bot bot left a comment

Choose a reason for hiding this comment

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

💡 To request another review, post a new comment with "/windsurf-review".

::
CPHASE01(theta) = [[1, 0, 0, 0],
CPHASE10(theta) = [[1, 0, 0, 0],
Copy link

Choose a reason for hiding this comment

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

Good catch! This correction properly aligns the gate name in the docstring with the actual implementation of the CPhase10Gate class.



def test_run__no_measurments(backend: RigettiQCSBackend):
def test_run__no_measurements(backend: RigettiQCSBackend):
Copy link

Choose a reason for hiding this comment

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

Good catch on fixing the spelling of 'measurements' in the function name. This improves code consistency and readability.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant