build(deps-dev): bump eslint-plugin-simple-import-sort from 10.0.0 to 13.0.0#1305
Conversation
Bumps [eslint-plugin-simple-import-sort](https://github.com/lydell/eslint-plugin-simple-import-sort) from 10.0.0 to 13.0.0. - [Changelog](https://github.com/lydell/eslint-plugin-simple-import-sort/blob/main/CHANGELOG.md) - [Commits](lydell/eslint-plugin-simple-import-sort@v10.0.0...v13.0.0) --- updated-dependencies: - dependency-name: eslint-plugin-simple-import-sort dependency-version: 13.0.0 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-simple-import-sort from version 10.0.0 to 13.0.0. This is a major version bump that introduces breaking changes, specifically a new deterministic ordering for imports (Namespace > Default > Named) and the removal of support for TypeScript import assignments (import A = require('module')).
Although Codacy marks the PR as up to standards, there is a significant implementation gap: no source files have been updated to align with the new sorting logic. It is highly likely that running the linter against the current codebase will result in errors. I recommend running the linter and committing any required style fixes before merging.
About this PR
- Verify that the codebase does not use TypeScript import assignments (e.g.,
import A = B.C), as support for this syntax was removed in version 12.0.0 and will now trigger linting errors. - The upgrade from v10.0.0 to v13.0.0 introduces breaking changes to how imports are sorted. Please verify that the codebase is still compliant by running
eslintoreslint --fix. If existing files are not updated, this PR will likely break the CI build upon merge.
Test suggestions
- Execute
eslint --fixon the codebase to verify and apply the new deterministic sort order for imports from the same source. - Verify that no TypeScript import assignments (
import A = B.C) exist in the codebase, as support was removed in version 12.0.0.
Prompt proposal for missing tests
Consider implementing these tests if applicable:
1. Execute `eslint --fix` on the codebase to verify and apply the new deterministic sort order for imports from the same source.
2. Verify that no TypeScript import assignments (`import A = B.C`) exist in the codebase, as support was removed in version 12.0.0.
TIP Improve review quality by adding custom instructions
TIP How was this review? Give us feedback
Bumps eslint-plugin-simple-import-sort from 10.0.0 to 13.0.0.
Changelog
Sourced from eslint-plugin-simple-import-sort's changelog.
Commits
90078e7eslint-plugin-simple-import-sort v13.0.07794d14Determinstic ordering between different import styles (#203)5ce648aFix deprecation warning when running tests27c4d1aFix code coveraged994f47Bump picomatch (#208)b8f246dBump flatted from 3.2.9 to 3.4.2 (#207)1c2d0e3Bump rollup from 4.50.1 to 4.59.0 (#206)23dd72fBump lodash from 4.17.21 to 4.17.23 (#204)0f8dc7aBump js-yaml from 4.1.0 to 4.1.1 (#202)4584b9eBump vite from 7.1.5 to 7.1.11 (#201)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)