sindresorhus/type-fest

Remove redundant code for compatibility with older versions of typescript

Open

#1.142 geöffnet am 16. Mai 2025

Auf GitHub ansehen
 (7 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)TypeScript (12.328 Stars) (471 Forks)batch import
enhancementgood first issue

Beschreibung

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>

Contributor Guide

Tech Stack
typescript
Domain
toolingdeveloper experience
Issue Type
refactor
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
under 1 hour
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
fresh
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
clear
Voraussetzungen
familiarity with TypeScript genericsunderstanding of type fest patterns
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
85
Research-Richtung
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.