-
Notifications
You must be signed in to change notification settings - Fork 13
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 ruff #497
add ruff #497
Conversation
45f0999
to
7c55b6f
Compare
print("2) Visit " + link) | ||
print("3) Enter code '" + code + "'") | ||
print("4) Enter your Microsoft credentials") | ||
print("To sign in:") # noqa: T201 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I was not fully sure which prints to keep, and which to remove. I think some of them might be helpful, but I removed some of them. Happy to discuss this and add the removed prints back in.
@@ -116,7 +118,7 @@ def get_detection_circuit(circuit: Circuit, n_device_qubits: int) -> Circuit: | |||
detection_circuit.Measure(qb, bb) | |||
postselection_qubits.append(qb) | |||
|
|||
for q in end_circuit_measures: | |||
for ps_b_index, q in enumerate(end_circuit_measures): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not sure how helpful this is, because there is still the second variable around?
|
||
line-length = 88 | ||
|
||
extend-exclude = ["examples"] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have excluded the examples notebooks, happy to add this later on is this would be helpful?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just some minor suggestions.
Co-authored-by: Alec Edgington <54802828+cqc-alec@users.noreply.github.com>
Thank you! Should be all updated now. |
Description
add ruff check to the CI
Related issues
Please mention any github issues addressed by this PR.
Checklist