-
Notifications
You must be signed in to change notification settings - Fork 2
fix: use overrideConfigFile: true + overrideConfig instead
#17
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
Conversation
🦋 Changeset detectedLatest commit: 984e232 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
WalkthroughA new changeset was added for the "@alauda/doom" package to document a patch update. The lint command's ESLint configuration was updated to use Changes
Sequence Diagram(s)sequenceDiagram
participant User
participant CLI (lint.ts)
participant ESLint
User->>CLI (lint.ts): Run lint command
CLI (lint.ts)->>ESLint: Initialize with overrideConfigFile: true and overrideConfig
CLI (lint.ts)->>ESLint: Provide TypeScript-specific config for ts/tsx files
ESLint-->>CLI (lint.ts): Lint results
CLI (lint.ts)-->>User: Output lint results
Poem
📜 Recent review detailsConfiguration used: CodeRabbit UI 📒 Files selected for processing (2)
🔇 Additional comments (3)
✨ Finishing Touches
🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR updates the ESLint configuration in the CLI lint command to use the new options overrideConfigFile and overrideConfig instead of the deprecated baseConfig. It also restores a previously commented block with type-checked linting rules for TypeScript and React.
- Replaces baseConfig with overrideConfig.
- Adds overrideConfigFile: true.
- Uncomments and restructures the TypeScript/React configuration block.
Comments suppressed due to low confidence (1)
src/cli/lint.ts:33
- Verify that replacing baseConfig with overrideConfig is fully aligned with the latest ESLint API guidelines to avoid any unintended configuration merging or behavior changes.
overrideConfig: tseslint.config([
Signed-off-by: JounQin <admin@1stg.me>
commit: |
Summary by CodeRabbit
Bug Fixes
Chores