import-js/eslint-plugin-import

Support for checking `require.resolve()`

Open

#585 aberto em 25 de set. de 2016

Ver no GitHub
 (2 comments) (4 reactions) (0 assignees)JavaScript (4.946 stars) (1.540 forks)batch import
acceptedenhancementhelp wanted

Description

Sometimes I use require.resolve() just to resolve paths to other resources than javascript modules.

Okay, this is not a regular import, but it would be nice if this plugin could also check these paths.

Maybe it could be a new setting like 'import/check-require-resolve': true, whereas various rules related to CommonJS imports also apply here.

Guia do colaborador

Pilha de tecnologia
javascriptnodejs
Domain
tooling
Tipo Issue
feature
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
half day
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
fresh
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
clear
Prerequisites
JavaScriptNode.js require.resolve()ESLint plugin developmenteslint plugin import structure
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
60
Direção de pesquisa
Examine how the plugin currently handles `require()` calls, particularly in rules like `no unresolved` and `no extraneous dependencies`. Identify where `require.resolve` is used as an expression and consider extending the AST visitor to detect `CallExpression` with callee `require.resolve`. Introduce a configuration option, e.g., `'import/check require resolve': true`, and apply the same import validation logic to the arguments of `require.resolve()`.