import-js/eslint-plugin-import

Suggestion [`no-named-default`]: add option to allow type imports with multiple imports

Open

#1.869 aberto em 5 de ago. de 2020

Ver no GitHub
 (6 comments) (4 reactions) (0 assignees)JavaScript (1.540 forks)batch import
bughelp wantedtypescript

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

Typescript's import type cannot be used to import the default export and named exports without using default as:

import type Foo, {Bar} from 'foo'
// A type-only import can specify a default import or named bindings, but not both.

However, these both work but trigger no-duplicates and no-named-default respectively:

import type Foo from 'foo'
import type {Bar} from 'foo'

import type {default as Foo, Bar} from 'foo'

Could there be a new option to no-named-default that allows type imports with multiple imports? For example:

import type {default as Foo} from 'foo' // lint warning
import type {default as Foo, Bar} from 'foo' // fine

Guia do colaborador

Direção de pesquisa
A issue solicita uma nova opção para a regra `no named default` para permitir importações de tipo com múltiplas importações. Examine o código fonte da regla no repositório, provavelmente em `src/rules/no named default.js`. Entenda como a regla atualmente processa as importações e identifique onde adicionar a lógica para instruções `import type`. Atualize a documentação da regla e adicione testes para a nova opção. Consulte as discussões vinculadas nos comentários da issue para esclarecimentos adicionais.
Pilha de tecnologia
typescriptjavascript
Domain
developer experiencetooling
Tipo Issue
Funcionalidade
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.
1-2 dias
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
TypeScriptESLint plugin development basics
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
45