feat: add rollup build pipeline with minification#73
Conversation
- Integrate rollup with terser for minified builds - Update CI to build before tests - Update publish workflow to build before release - Exclude dist/ from git, publish only built output - Configure biome to ignore generated dist folder
|
Warning Rate limit exceeded
⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (18)
Note Reviews pausedIt looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the Use the following commands to manage reviews:
Use the checkboxes below for quick actions:
📝 WalkthroughWalkthroughAdds a Rollup bundling pipeline and switches CLI entrypoints to use Changes
Sequence Diagram(s)sequenceDiagram
participant Dev as Developer
participant CI as CI Runner
participant Build as Rollup / Package Builder
participant Smoke as Smoke Test
participant Store as Artifact Store
Dev->>CI: push / open PR
CI->>CI: install dependencies
CI->>Build: run build (matrix package-manager run build / pnpm build)
Build-->>CI: produce `dist/` artifacts
CI->>Smoke: run smoke test `node dist/create-next-quick.js --version`
Smoke-->>CI: write `test-logs/smoke-test.txt` (stdout/stderr + exit code)
alt smoke test success
CI->>Store: upload logs & artifacts
else smoke test failure
CI-->>Dev: fail job
end
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
✅ CI/CD Test ResultsOverall Status: success |
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
- Add rollup with terser for minified production builds - Rename entry point to create-next-quick.js - Update tests to run against built dist artifact - Add CI smoke test step to validate built artifact - Remove unused @rollup/plugin-alias dependency - Fix biome config (remove invalid files.ignore key)
…ai/create-next-quick into feat/add-build-pipeline
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
This comment was marked as outdated.
This comment was marked as outdated.
Fixed 1 file(s) based on 1 unresolved review comment. Co-authored-by: CodeRabbit <noreply@coderabbit.ai>
…ai/create-next-quick into feat/add-build-pipeline
Summary by CodeRabbit
Tests
Documentation
Chores
Repository