chore: A way to skip checks before publishing, if we know tests already pass#2281
chore: A way to skip checks before publishing, if we know tests already pass#2281
Conversation
|
pkg.pr.new packages benchmark commit |
📊 Bundle Size Comparison
👀 Notable resultsStatic test results:No major changes. Dynamic test results:No major changes. 📋 All resultsClick to reveal the results table (348 entries).
If you wish to run a comparison for other, slower bundlers, run the 'Tree-shake test' from the GitHub Actions menu. |
There was a problem hiding this comment.
Pull request overview
Adds an environment-variable based escape hatch to skip prepublish checks when publishing packages (useful when tests are already known to be green), and documents it in the release checklist.
Changes:
- Introduce
SKIP_ALL_CHECKS=trueas an alternative to the--skip-all-checksflag intgpu-dev-cli prepack. - Reuse a single
skipAllChecksboolean for both publish-tag verification and post-build checks. - Document the new environment variable in
CONTRIBUTING.md.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| packages/tgpu-dev-cli/prepack.mjs | Adds env-var support for skipping checks and wires it into existing gating logic. |
| CONTRIBUTING.md | Documents a publishing tip to skip checks via SKIP_ALL_CHECKS=true. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
74484b6 to
d0eb7d8
Compare
aleksanderkatan
left a comment
There was a problem hiding this comment.
Are we sure we want this? How does our publishing pipeline work, are we running the CI twice?
d0eb7d8 to
1367f78
Compare
1367f78 to
cb20bac
Compare
No description provided.