Cirq icon indicating copy to clipboard operation
Cirq copied to clipboard

Only use `chr(30)` and `chr(31)` delimiters in cirq-ionq

Open splch opened this issue 2 months ago • 3 comments

  • [x] update tests to remove\u* and \x*

relates to #5216

splch avatar Oct 10 '25 00:10 splch

Codecov Report

:white_check_mark: All modified and coverable lines are covered by tests. :white_check_mark: Project coverage is 99.38%. Comparing base (8dd8a8f) to head (01c892c).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #7699      +/-   ##
==========================================
- Coverage   99.38%   99.38%   -0.01%     
==========================================
  Files        1090     1090              
  Lines       98300    98301       +1     
==========================================
  Hits        97695    97695              
- Misses        605      606       +1     

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

:rocket: New features to boost your workflow:
  • :snowflake: Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • :package: JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

codecov[bot] avatar Oct 10 '25 02:10 codecov[bot]

@splch Thank you for your work on this! When you get a chance, could you update the title of this PR to mention that this is for cirq-ionq, and also add a bit more explanation to the PR description? This will help future contributors and maintainers when reading and searching through past work.

mhucka avatar Oct 13 '25 23:10 mhucka

@splch I'm not sure if this draft PR is still something planned. It looks like the associated issue was closed. Maybe this should be closed too? (No biggie either way; just going around and checking up on things.)

mhucka avatar Nov 23 '25 03:11 mhucka