sinonjs/sinon

Use knip instead of unimported

Open

#2 637 ouverte le 10 févr. 2025

Voir sur GitHub
 (2 commentaires) (0 réactions) (1 assigné)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.

Guide contributeur

Stack technique
javascriptnodejs
Domaine
toolingbuild system
Type d'issue
feature
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
half day
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
clear
Prérequis
Node.jsJavaScriptsinon project structure
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
65
Direction de recherche
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.