import-js/eslint-plugin-import

Defining internal-regex and import/external-module-folders together yields unexpected errors

Open

#2.139 aberto em 25 de jun. de 2021

Ver no GitHub
 (2 comments) (0 reactions) (0 assignees)JavaScript (1.540 forks)batch import
bughelp wanted

Métricas do repositório

Stars
 (4.946 stars)
Métricas de merge de PR
 (Mesclagem média 138d 22h) (3 fundiu PRs em 30d)

Description

In our monorepo we store our internal packages under ./packages

So we tried a config like this:

rules: {
  'import/order': 'warn',
},
settings: {
  'import/internal-regex': '^internal-', //  This makes importing monorepo packages get treated as though they were 'relative-parent-imports'
  'import/external-module-folders': ['node_modules', 'packages'],
}

But that makes a non-relative import throw this unexpected error

import { foo } from 'internal-utils';
                     ^--- // Relative imports from parent directories are not allowed.

Guia do colaborador

Direção de pesquisa
Investigue o código fonte do plugin para entender como 'internal regex' e 'external module folders' interagem. Examine a implementação da regra 'import/order', especificamente como os tipos de importação são resolvidos. Verifique se há issues ou PRs relacionados que abordam comportamentos semelhantes.
Pilha de tecnologia
javascriptnodejs
Domain
developer experiencetooling
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
ESLintNode.js module system
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
60