Skip to content

Conversation

pavoljuhas
Copy link
Collaborator

@pavoljuhas pavoljuhas commented Sep 24, 2025

  • Change test_qcircuit_pdf to compile PDF file
  • Install LaTeX and latexmk for the CI jobs

Prerequisite for tests in #7354

@github-actions github-actions bot added the size: S 10< lines changed <50 label Sep 24, 2025
Copy link

codecov bot commented Sep 24, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 99.37%. Comparing base (b1f8b82) to head (f66e269).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #7665      +/-   ##
==========================================
- Coverage   99.37%   99.37%   -0.01%     
==========================================
  Files        1082     1082              
  Lines       96691    96691              
==========================================
- Hits        96086    96084       -2     
- Misses        605      607       +2     

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

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

@pavoljuhas pavoljuhas marked this pull request as ready for review September 24, 2025 21:25
@pavoljuhas pavoljuhas requested review from a team and vtomole as code owners September 24, 2025 21:25
@pavoljuhas
Copy link
Collaborator Author

NB: The installation of texlive adds about 1 minute to the duration of affected CI jobs.

@pavoljuhas pavoljuhas added this pull request to the merge queue Sep 24, 2025
Merged via the queue into quantumlib:main with commit 9550c0d Sep 24, 2025
35 checks passed
@pavoljuhas pavoljuhas deleted the ci-test-with-latexmk branch September 24, 2025 22:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size: S 10< lines changed <50

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants