import-js/eslint-plugin-import

no-restricted-paths - absolute path expected where relative path is provided

Open

#1.890 aberto em 24 de ago. de 2020

Ver no GitHub
 (1 comment) (0 reactions) (0 assignees)JavaScript (1.540 forks)batch import
help wanted

Métricas do repositório

Stars
 (4.946 stars)
Métricas de merge de PR
 (Mesclagem média 5d 21h) (9 fundiu PRs em 30d)

Description

I am having trouble with the configuration for this rule and believe there is a bug in the way paths are resolved

// see https://github.com/benmosher/eslint-plugin-import/blob/master/src/rules/no-restricted-paths.js#L72
// it returns a relative path, are we sure thats what we want? 
const absoluteImportPath = resolve(importPath, context)

Here we use resolve which comes from utils/resolve and it returns a relative path

https://github.com/benmosher/eslint-plugin-import/blob/569d72688fa6ae5c038c51eafa4f6016fc1ee802/utils/resolve.js#L220

should we be using path.resolve to return an absolute path instead? I am happy to open a PR with the change once confirmed

to be clear the change I am proposing is

- const absoluteImportPath = resolve(basePath, context)
+ const absoluteImportPath = path.resolve(basePath, importPath);

Guia do colaborador

Direção de pesquisa
Examine os arquivos src/rules/no restricted paths.js linha 72 e utils/resolve.js linha 220. Confirme se a resolução do caminho de importação deve ser absoluta verificando o comportamento esperado da regra. Se confirmado, altere a chamada resolve para path.resolve, então execute os testes para garantir que não haja regressão. Verifique também se outras regras usam chamadas resolve semelhantes que possam precisar da mesma correção.
Pilha de tecnologia
javascriptnodejs
Domain
toolingdeveloper experience
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.
Antigo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
Basic knowledge of Node.js path moduleFamiliarity with ESLint plugin rules
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
70