Skip to content

Conversation

pavoljuhas
Copy link
Collaborator

Avoid [unused-ignore] error from check/mypy when qsimcirq is installed.

Avoid `[unused-ignore]` error when qsimcirq is installed.
@pavoljuhas pavoljuhas requested review from wcourtney, vtomole, verult and a team as code owners February 13, 2025 01:29
@CirqBot CirqBot added the Size: XS <10 lines changed label Feb 13, 2025
Copy link

codecov bot commented Feb 13, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 98.17%. Comparing base (dd3df78) to head (f72f194).
Report is 2 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #7063      +/-   ##
==========================================
- Coverage   98.18%   98.17%   -0.01%     
==========================================
  Files        1089     1089              
  Lines       95208    95208              
==========================================
- Hits        93478    93474       -4     
- Misses       1730     1734       +4     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@pavoljuhas pavoljuhas requested a review from mhucka February 13, 2025 03:54
Copy link
Contributor

@mhucka mhucka left a comment

Choose a reason for hiding this comment

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

LGTM.

@pavoljuhas pavoljuhas added this pull request to the merge queue Feb 13, 2025
Merged via the queue into quantumlib:main with commit d5c2b41 Feb 13, 2025
38 checks passed
@pavoljuhas pavoljuhas deleted the mypy-with-and-without-qsimcirq branch February 13, 2025 23:21
BichengYing pushed a commit to BichengYing/Cirq that referenced this pull request Jun 20, 2025
…7063)

Avoid `[unused-ignore]` error when qsimcirq is installed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Size: XS <10 lines changed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants