Skip to content

chore: willboosterify this repo#754

Open
WillBooster-bot wants to merge 1 commit intomainfrom
wbfy
Open

chore: willboosterify this repo#754
WillBooster-bot wants to merge 1 commit intomainfrom
wbfy

Conversation

@WillBooster-bot
Copy link
Copy Markdown
Contributor

No description provided.

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request updates the @willbooster/wb dependency to version 13.12.14 across the monorepo, adds @types/* to the preapproved packages in .yarnrc.yml, and refactors Oxlint configurations to properly handle root-only options in package-local files. Feedback was provided to simplify the root Oxlint configuration by removing a redundant variable assignment and importing the base configuration directly with the desired name.

Comment thread oxlint.config.ts
Comment on lines +2 to +4
import oxlintBaseConfig from '@willbooster/oxlint-config';

const oxlintResolvedConfig = oxlintBaseConfig;
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

This assignment is redundant. You can simplify the code by importing the configuration directly with the desired name, adhering to the rule of eliminating redundancy.

Suggested change
import oxlintBaseConfig from '@willbooster/oxlint-config';
const oxlintResolvedConfig = oxlintBaseConfig;
import oxlintResolvedConfig from '@willbooster/oxlint-config';
References
  1. Simplify code as much as possible to eliminate redundancy. (link)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant