import-js/eslint-plugin-import

[import/no-duplicates] `{"prefer-inline": true}` does not work

Open

#2 715 ouverte le 12 févr. 2023

Voir sur GitHub
 (14 commentaires) (13 réactions) (0 assignés)JavaScript (4 946 stars) (1 540 forks)batch import
bughelp wantedtypescript

Description

version 2.27.5

https://github.com/import-js/eslint-plugin-import/blob/main/docs/rules/no-duplicates.md#inline-type-imports The documentation states that adding {"prefer-inline": true} as shown below will cause errors and autofixes if there are duplicate imports. However, this was not possible in my environment.

{
  "import/no-duplicates": ["error", {"prefer-inline": true}],
}

It should look like the following, but there is no error or anything.

import NextErrorComponent from 'next/error';
import { type ErrorProps } from 'next/error';
// ↓
import NextErrorComponent, { type ErrorProps } from 'next/error';

image

Is this currently being corrected?

Guide contributeur

Stack technique
javascripttypescriptnodejs
Domaine
toolingdeveloper experience
Type d'issue
bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 hours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
clear
Prérequis
Basic ESLint configurationTypeScript import syntax
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70
Direction de recherche
Review the `no duplicates` rule implementation in the eslint plugin import repository, specifically how the `prefer inline` option is handled. Look at the rule's source file (likely `src/rules/no duplicates.js`) and its tests. Check if there are any existing PRs or issues addressing this. The documentation mentions inline type imports, but the actual behavior may not merge duplicate imports as expected. Focus on the logic that merges imports and how the option affects it.