sindresorhus/type-fest

Remove redundant code for compatibility with older versions of typescript

Open

#1 142 ouverte le 16 mai 2025

Voir sur GitHub
 (7 commentaires) (0 réactions) (0 assignés)TypeScript (12 328 stars) (471 forks)batch import
enhancementgood first issue

Description

In some previously supported but now unsupported versions of typescript, there would be some incorrect circular check, and in order to bypass the circular check, we have to add some non-essential code, for example:

type A<T> = 
 SomeToolType<T> extends infer B 
  ? B
  : never // Never happen

The above code can be simplified now:

type A<T> = SomeToolType<T>

Guide contributeur

Stack technique
typescript
Domaine
toolingdeveloper experience
Type d'issue
refactor
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
under 1 hour
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
fresh
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
clear
Prérequis
familiarity with TypeScript genericsunderstanding of type fest patterns
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
85
Direction de recherche
Search the codebase for patterns like `SomeToolType<T> extends infer B ? B : never` where the conditional type is unnecessary due to removed support for older TypeScript versions. Look in source type definitions for such redundant patterns and simplify them to directly use `SomeToolType<T>`. The issue provides an example; apply similar logic to other occurrences. Check the repository's CONTRIBUTING.md for contribution guidelines.