Skip to content

Fix docs path in snippet test and exclude subdirs#7798

Merged
pavoljuhas merged 5 commits intoquantumlib:mainfrom
ToastCheng:i7787
Dec 11, 2025
Merged

Fix docs path in snippet test and exclude subdirs#7798
pavoljuhas merged 5 commits intoquantumlib:mainfrom
ToastCheng:i7787

Conversation

@ToastCheng
Copy link
Copy Markdown
Contributor

@ToastCheng ToastCheng commented Dec 9, 2025

  1. Fix docs path in snippet test and exclude md under hardware and google
  2. Fix exec error in md files

Partially implements #7787

@ToastCheng ToastCheng requested review from a team and vtomole as code owners December 9, 2025 01:23
@ToastCheng ToastCheng requested a review from fdmalone December 9, 2025 01:23
@github-actions github-actions bot added the Size: XS <10 lines changed label Dec 9, 2025
@ToastCheng ToastCheng force-pushed the i7787 branch 3 times, most recently from ec4b094 to 08b2d49 Compare December 9, 2025 16:19
@github-actions github-actions bot added size: S 10< lines changed <50 and removed Size: XS <10 lines changed labels Dec 9, 2025
@codecov
Copy link
Copy Markdown

codecov bot commented Dec 9, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 99.57%. Comparing base (a764b67) to head (6c17e8a).
⚠️ Report is 4 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #7798   +/-   ##
=======================================
  Coverage   99.57%   99.57%           
=======================================
  Files        1102     1102           
  Lines       98434    98529   +95     
=======================================
+ Hits        98014    98109   +95     
  Misses        420      420           

☔ 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.

@ToastCheng ToastCheng changed the title Fix docs folder path in snippet test Fix docs path in snippet test and exclude subdirs Dec 9, 2025
Since test_can_run_docs_code_snippets will run the snippets
in markdown, fix the undefined error in the snippets.
Copy link
Copy Markdown
Collaborator

@pavoljuhas pavoljuhas left a comment

Choose a reason for hiding this comment

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

LGTM after pushing a few tweaks, mainly to avoid creating scratch test file from testing of serialization.md.

@pavoljuhas pavoljuhas added this pull request to the merge queue Dec 11, 2025
Merged via the queue into quantumlib:main with commit 9e2123d Dec 11, 2025
37 checks passed
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