fix(ci): land npm-dev updates on TypeScript 6 - #46
Conversation
Dependabot's grouped npm-dev PR failed lint because typescript-eslint does not support TypeScript 7.0 (no compiler API until 7.1). Pin TypeScript to 6.0.3 and apply the rest of the toolchain bumps. Silence tsup's injected baseUrl deprecation only in the DTS pass (egoist/tsup#1388). Keep react-hooks on rules-of-hooks; v7 recommended also enables compiler rules that flag intentional render-time ref writes. Co-authored-by: dojce1048 <dojce1048@gmail.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
@coderabbitai full review |
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
|
Warning Review the following alerts detected in dependencies. According to your organization's Security Policy, it is recommended to resolve "Warn" alerts. Learn more about Socket for GitHub.
|
|
Warning Review limit reachedNext included review available in 55 minutes. View limit detailsLimit details: You’ve used the included review currently available. Your 95 included PR review attempts over the past 7 days set your current allowance at 1 review per hour. Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available. Review configuration: ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Advanced Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (4)
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Advanced Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (4)
Included review availability: 0 reviews are currently available. Your included PR review attempts over the past 7 days set your current allowance at 1 review per hour. 📝 WalkthroughWalkthroughThe pull request updates ESLint React Hooks rules, configures TypeScript declaration builds to ignore TypeScript 6 deprecations, and updates root and website development dependencies. ChangesTooling updates
Priority: ⬇️ Low Estimated code review effort: 2 (Simple) | ~10 minutes Merge Risk: ⚪ Minimal · up to This updates development tooling, keeps the intended Hooks lint enforcement, and scopes the TypeScript deprecation suppression to declaration builds. No current merge-blocking risk is identified. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches 💡 1🛠️ Fix failing CI checks 💡
📝 Generate docstrings
🧪 Generate unit tests (beta)
A rabbit reads each line, Comment |
|
Dependabot's grouped npm-dev PR failed lint because typescript-eslint does not support TypeScript 7.0 (no compiler API until 7.1). Pin TypeScript to 6.0.3 and apply the rest of the toolchain bumps.
Silence tsup's injected baseUrl deprecation only in the DTS pass (egoist/tsup#1388). Keep react-hooks on rules-of-hooks; v7 recommended also enables compiler rules that flag intentional render-time ref writes.
Summary by CodeRabbit