build(deps-dev): bump eslint-plugin-unused-imports from 3.0.0 to 4.4.1#1303
build(deps-dev): bump eslint-plugin-unused-imports from 3.0.0 to 4.4.1#1303dependabot[bot] wants to merge 1 commit into
Conversation
Bumps [eslint-plugin-unused-imports](https://github.com/sweepline/eslint-plugin-unused-imports) from 3.0.0 to 4.4.1. - [Release notes](https://github.com/sweepline/eslint-plugin-unused-imports/releases) - [Commits](https://github.com/sweepline/eslint-plugin-unused-imports/commits/v4.4.1) --- updated-dependencies: - dependency-name: eslint-plugin-unused-imports dependency-version: 4.4.1 dependency-type: direct:development update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com>
Up to standards ✅🟢 Issues
|
| Metric | Results |
|---|---|
| Complexity | 0 |
| Duplication | 0 |
AI Reviewer: first review requested successfully. AI can make mistakes. Always validate suggestions.
TIP This summary will be updated as you push new changes.
There was a problem hiding this comment.
Pull Request Overview
This PR upgrades eslint-plugin-unused-imports from v3 to v4.4.1. While Codacy results are up to standards, this major version jump involves a significant architecture change: the removal of eslint-rule-composer. Consequently, the unused-imports/no-unused-vars rule no longer inherits settings from the base ESLint no-unused-vars rule. You must verify if custom linting configurations (e.g., ignore patterns) need to be manually duplicated to avoid inconsistencies. Additionally, the PR description lacks the v4.0.0 changelog, which is the source of these breaking changes.
About this PR
- This is a major version upgrade (v3.x to v4.x). The current PR description omits the breaking changes introduced in the v4.0.0 release. It is critical to ensure that peer dependencies (ESLint and TypeScript-ESLint) and the removal of
eslint-rule-composerin the lockfile align with the project's requirements.
Test suggestions
- Execute the project's linting suite to verify that the upgrade does not introduce breaking changes or configuration errors.
- Confirm that the
unused-imports/no-unused-importsrule correctly identifies and cleans up unused imports in TypeScript files.
Prompt proposal for missing tests
Consider implementing these tests if applicable:
1. Execute the project's linting suite to verify that the upgrade does not introduce breaking changes or configuration errors.
2. Confirm that the `unused-imports/no-unused-imports` rule correctly identifies and cleans up unused imports in TypeScript files.
TIP Improve review quality by adding custom instructions
TIP How was this review? Give us feedback
| "@types/simple-mock": "^0.8.6", | ||
| "eslint-plugin-simple-import-sort": "^10.0.0", | ||
| "eslint-plugin-unused-imports": "^3.0.0", | ||
| "eslint-plugin-unused-imports": "^4.4.1", |
There was a problem hiding this comment.
🟡 MEDIUM RISK
This major version bump (v3 to v4) introduces a rewrite that stops the unused-imports/no-unused-vars rule from automatically inheriting settings from the base ESLint no-unused-vars rule. You must manually sync any custom configurations to the new rule. Try running the following prompt in your coding agent: > Update the ESLint configuration to explicitly define options for unused-imports/no-unused-vars, mirroring any existing settings in the base no-unused-vars rule.
Bumps eslint-plugin-unused-imports from 3.0.0 to 4.4.1.
Release notes
Sourced from eslint-plugin-unused-imports's releases.
Commits
Maintainer changes
This version was pushed to npm by GitHub Actions, a new releaser for eslint-plugin-unused-imports since your current version.
Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting
@dependabot rebase.Dependabot commands and options
You can trigger Dependabot actions by commenting on this PR:
@dependabot rebasewill rebase this PR@dependabot recreatewill recreate this PR, overwriting any edits that have been made to it@dependabot show <dependency name> ignore conditionswill show all of the ignore conditions of the specified dependency@dependabot ignore this major versionwill close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this minor versionwill close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this dependencywill close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)