Skip to content
This repository was archived by the owner on Nov 4, 2025. It is now read-only.

chore: Remove unused dependencies and add missing dependencies #303

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

chore: Remove unused dependencies and add missing dependencies #303

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion package.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -48,7 +48,7 @@
"@types/react": "^18.0.0",
"@types/react-dom": "^18.0.0",
"@types/warning": "^3.0.0",
"cross-env": "^7.0.3",
"@umijs/fabric": "^4.0.1",
Copy link

@gemini-code-assist gemini-code-assist bot Nov 4, 2025

Choose a reason for hiding this comment

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

high

Adding @umijs/fabric introduces a potential dependency version conflict. This package depends on eslint-plugin-unicorn@^42.0.0, while this project explicitly requires eslint-plugin-unicorn@^55.0.0 (line 54). This could result in two different versions of the plugin being installed, potentially causing inconsistent linting behavior or errors.

To resolve this, you could consider one of the following approaches:

  • Remove eslint-plugin-unicorn from devDependencies and rely on the version from @umijs/fabric, if its features are sufficient.
  • If version 55 is required, you could use overrides in package.json (for npm) or resolutions (for Yarn) to enforce a single version of eslint-plugin-unicorn, after ensuring compatibility.

"dumi": "^2.0.15",
"eslint": "^8.56.0",
"eslint-plugin-unicorn": "^55.0.0",
Expand Down
Loading