import-js/eslint-plugin-import

no-extraneous-dependencies: autofix

Open

#248 aperta il 19 apr 2016

Vedi su GitHub
 (3 commenti) (6 reazioni) (0 assegnatari)JavaScript (1540 fork)batch import
enhancementhelp wantedsemver-minor

Metriche repository

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

Descrizione

My expectations for this are:

  1. it must be configured explicitly via some rule config option, since it makes external modifications
    • i.e., if you use autofix and this rule, it won't attempt installs unless you've enabled it explicitly
  2. it calls npm install [package] vs. mutating the package.json directly
    • as such, if it is extraneous and you haven't installed it, you get all the dependencies
  3. aforementioned rule config is something like install: 'SE' meaning the fix call is npm i [dep] -SE.

I can imagine changing position on (1) and (3) on account of .npmrc existing. Perhaps its existence qualifies as "explicit config".

Note that this means that if your editor has linter integration and autofix enabled on save, you could npm install simply by adding an import/require statement to code.

Would be pretty sweet. Essentially an alternate ESLint-powered form of npm-install-webpack-plugin at that point.

cc-ing @jfmengels as author of no-extra-deps and @sindresorhus for +1-ing my comment about this on the original rule issue.

Guida contributor

Direzione di ricerca
Esamina l'implementazione corrente della regola no extraneous dependencies e comprendi come viene implementata la correzione automatica in altre regole dello stesso plugin. Progetta un'opzione di configurazione per abilitare la funzionalità di correzione automatica e implementa la logica per chiamare npm install con i flag appropriati. Testa la correzione automatica con vari scenari.
Tech stack
javascript
Dominio
toolingdeveloper experience
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-2 giorni
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Abbastanza chiara
Prerequisiti
ESLintnpm
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
40