Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ci: fix documentation coverage check #8907

Merged
merged 10 commits into from
Jan 2, 2025
Merged

ci: fix documentation coverage check #8907

merged 10 commits into from
Jan 2, 2025

Conversation

chenjiahan
Copy link
Member

Summary

The documentation coverage check CI has been skipped since #8072.

This PR added it back and use tsx to ensure it can get the zod meta data from the source code.

Checklist

  • Tests updated (or not required).
  • Documentation updated (or not required).

@github-actions github-actions bot added the team The issue/pr is created by the member of Rspack. label Dec 31, 2024
Copy link

netlify bot commented Dec 31, 2024

Deploy Preview for rspack canceled.

Name Link
🔨 Latest commit cd5f416
🔍 Latest deploy log https://app.netlify.com/sites/rspack/deploys/677620402c5c3e0008c575bb

Copy link

codspeed-hq bot commented Dec 31, 2024

CodSpeed Performance Report

Merging #8907 will not alter performance

Comparing doc_check_ci_1231 (cd5f416) with main (46e8278)

Summary

✅ 3 untouched benchmarks

SyMind
SyMind previously approved these changes Jan 2, 2025
@chenjiahan chenjiahan requested a review from SyMind January 2, 2025 05:47
@chenjiahan chenjiahan merged commit d2721e9 into main Jan 2, 2025
32 checks passed
@chenjiahan chenjiahan deleted the doc_check_ci_1231 branch January 2, 2025 06:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
team The issue/pr is created by the member of Rspack.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants