Fix while_loop hint, update pragma hint in visitor.py - #371
Conversation
Updated type hints for method arguments and return types in visitor.py.
Argus reviewAuto-review is off for this repo. Tick the box below to run a review on this PR.
Estimated cost
Tip: you can also comment |
|
👋 Hey there! It looks like the changelog might need an update. Please take a moment to edit the
|
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
|
||
| Args: | ||
| statement (qasm3_ast.Pragma): The Pragma node to visit. | ||
| statement (Pragma): The Pragma node to visit. |
TheGupta2012
left a comment
There was a problem hiding this comment.
Added one comment, but should be okay to merge
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
Summary of changes
Fixes a mistake introduced by #346, which claims that
visitor:_visit_while_loopis affected byself._check_only.Additionally, updates the docstring of
visitor:_visit_pragmato match the rest of the file.