sinonjs/sinon

Use knip instead of unimported

Open

#2.637 aberto em 10 de fev. de 2025

Ver no GitHub
 (2 comments) (0 reactions) (1 assignee)JavaScript (9.548 stars) (764 forks)batch import
Help wantedgood first issue

Description

In https://github.com/sinonjs/sinon/pull/2636 we removed unimported, because it had been archived.

The author recommends using knip instead.

Let's implement knip and use it for finding unused files and dependencies.

Guia do colaborador

Pilha de tecnologia
javascriptnodejs
Domain
toolingbuild system
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.
active
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
clear
Prerequisites
Node.jsJavaScriptsinon project structure
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
65
Direção de pesquisa
Review PR #2636 for context on the previous unimported setup. Create a knip configuration file (e.g., knip.json) and add a npm script to run knip. Update the project's CI configuration if needed to include knip checks.