Galooshi/import-js

ImportJS misidentifies `import` methods to be Dynamic Import

Open

#529 aperta il 18 mar 2019

Vedi su GitHub
 (3 commenti) (0 reazioni) (0 assegnatari)JavaScript (70 fork)batch import
bughelp wanted

Metriche repository

Star
 (519 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

I just run ImportJS on a huge codebase and the results are mostly good, but this one seemed peculiar enough for me to report it:

I'm not too familiar with the lib, but in this case it broke the code by sorting two variable assignments in false order.

Used ImportJS 3.2.0 on Mac OS, via CLI:

find ./client -name "**.js*" -exec importjs fix --overwrite {} \;

//.importjs.js
module.exports = {
    declarationKeyword: 'import',
    environments: ['meteor', 'node'],
    excludes: ['.meteor/local/**', '.vscode/**'],
    mergableOptions: {
        globals: false,
    },
    useRelativePaths: true
}

Guida contributor

Direzione di ricerca
Indaga la logica di parsing in ImportJS che distingue le importazioni statiche da quelle dinamiche. Cerca come lo strumento identifica le espressioni `import()` rispetto alle dichiarazioni statiche `import`, poiché il bug mostra che classifica erroneamente le chiamate a metodi chiamati 'import' come importazioni dinamiche. Controlla l'attraversamento dell'AST o i pattern regex utilizzati.
Tech stack
javascriptnodejs
Dominio
clitooling
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
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
GitNode.js
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70