import-js/eslint-plugin-import

In `import/order`, modules starting with `@/` are always treated as `unknown`

Open

#2249 aperta il 7 ott 2021

Vedi su GitHub
 (3 commenti) (6 reazioni) (0 assegnatari)JavaScript (1540 fork)batch import
help wantedimport/export ordering

Metriche repository

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

Descrizione

If you run yarn lint on https://github.com/ykzts-sandbox/nextjs-with-module-path-aliases, you will get the following output.

$ yarn lint
yarn run v1.22.11
$ next lint
info  - Using webpack 5. Reason: Enabled by default https://nextjs.org/docs/messages/webpack5

./pages/index.tsx
5:1  Error: `../styles/Home.module.css` import should occur before import of `@/components/hello`  import/order

info  - Need to disable some ESLint rules? Learn more here: https://nextjs.org/docs/basic-features/eslint#disabling-rules
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

regression from #2146

Guida contributor

Direzione di ricerca
Inizia esaminando la PR #2146, che ha introdotto la regressione. Guarda il file `rules/order.js` per capire come vengono determinati i tipi di import, specialmente per gli alias di percorso. Il problema fornisce un repository di riproduzione, quindi clonalo ed esegui il linter per vedere l'errore. Indaga la logica che classifica le import come 'unknown' e modificala per riconoscere correttamente i percorsi `@/`.
Tech stack
javascriptnodejs
Dominio
tooling
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
Node.jsESLint
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
65