Conversation
|
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: defaults Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (6)
📝 WalkthroughWalkthroughProject version bumped from 3.3.3 to 3.3.5, Changes
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~25 minutes Possibly related PRs
Poem
✨ 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 |
There was a problem hiding this comment.
Pull request overview
Release bump to 3.3.5, updating internal tooling to be a bit more efficient (less unnecessary writes) and speeding up lint runs via ESLint caching.
Changes:
- Refactors build-time Prettier formatting into reusable helpers and avoids rewriting files when unchanged.
- Enables ESLint caching in both the build script and npm scripts (with a custom cache location).
- Updates package versions/lockfile for the 3.3.5 release.
Reviewed changes
Copilot reviewed 5 out of 7 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| src/build/build-prettier.ts | Extracts Prettier option resolution + format/check into helper functions and reduces redundant writes. |
| src/build/build-eslint.ts | Enables ESLint cache with a custom cache location/strategy. |
| src/build/build-distfiles.ts | Avoids rewriting dist files when content is unchanged. |
| pnpm-lock.yaml | Locks updated dependency graph for the release. |
| package.json | Version bump + dependency bumps + lint scripts updated to use caching. |
| files/empty-project/package.json | Updates template devDependency versions to match release. |
| .gitignore | Adds an ignore entry for eslint cache (but does not match new cache location). |
Files not reviewed (1)
- pnpm-lock.yaml: Language not supported
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| const eslint = new ESLint({ | ||
| fix: !ci, | ||
| cache: true, | ||
| cacheLocation: ".cache/eslint/.eslintcache", |
There was a problem hiding this comment.
cacheLocation is set to .cache/eslint/.eslintcache, but the PR only adds .eslintcache to .gitignore. Without also ignoring .cache/ (or .cache/eslint/), running lint locally will likely create untracked files under .cache/ in the repo.
| cacheLocation: ".cache/eslint/.eslintcache", | |
| cacheLocation: ".eslintcache", |
Summary by CodeRabbit
Chores
Refactor