guardian/dotcom-rendering

Remove `index.d.ts`

Aberta

#7.638 aberto em 26 de abr. de 2023

Ver no GitHubGuia do colaborador
 (3 comentários) (1 reação) (1 responsável)TypeScript (35 forks)auto 404
Pick me upTeam Candidate: WebXgood first issue

Métricas do repositório

Stars
 (271 estrelas)
Métricas de merge de PR
 (Métricas PR pendentes)

Description

Why?

Declaration files create ambient types, which appear in a global namespace across the project. We think it's preferable to avoid adding types to a global scope, and instead prefer to import them as needed.

How?

Migrate the types in the index.d.ts1 file into normal .ts files, and import them as needed:

Footnotes

  1. https://github.com/guardian/dotcom-rendering/blob/4ea272b55efa01ba88f0f5ff60f6bc9d14198147/dotcom-rendering/index.d.ts ↩

Guia do colaborador

Direção de pesquisa
Identifique todos os tipos em index.d.ts e mova os para arquivos .ts separados e importe os onde necessário.
Pilha de tecnologia
typescript
Domain
frontend
Tipo Issue
Refatoração
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
TypeScript
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
75