import-js/eslint-plugin-import

[import/no-duplicates] Extra comma added when fixer is run

Open

#2757 aperta il 9 apr 2023

Vedi su GitHub
 (0 commenti) (2 reazioni) (0 assegnatari)JavaScript (1540 fork)batch import
bughelp wanted

Metriche repository

Star
 (4946 star)
Metriche merge PR
 (Merge medio 138g 22h) (3 PR mergiate in 30 g)

Descrizione

Input:

import { Foo } from "someImport";
import {
  BarLongNameToEnsureLineBreak,
  BazManLongNameToEnsureLineBreak,
} from "someImport";
import { Qux } from "someImport";

const foo: Foo = "foo";
const bar: BarLongNameToEnsureLineBreak = "bar";
const baz: BazManLongNameToEnsureLineBreak = "baz";
const qux: Qux = "qux";

output:

import { Foo ,
  BarLongNameToEnsureLineBreak,
  BazManLongNameToEnsureLineBreak,
, Qux } from "someImport";

const foo: Foo = "foo";
const bar: BarLongNameToEnsureLineBreak = "bar";
const baz: BazManLongNameToEnsureLineBreak = "baz";
const qux: Qux = "qux";

Note the commas at the end of line 3 and start of line 4.

Possibly related to #2601, tho in that case it appears that the opposite problem is occurring: no comma when there should be

Guida contributor

Direzione di ricerca
Ispeziona la logica del fixer nella regola no duplicates. Guarda come la regola unisce le dichiarazioni di importazione e gestisce le virgole, specialmente quando ci sono interruzioni di riga. Traccia il percorso del codice che genera la virgola extra.
Tech stack
javascript
Dominio
backendtooling
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
ESLintJavaScript
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
50