microsoft / microsoft/TypeScript
Add readonly modifier to Math.min and Math.max
Ouverte
Personne n'a encore pris cette issue.
Bug
Domain: check: Type Inference
- Langage dominant
- Go
- Étoiles
- 111k
- Forks
- 14.3k
- Merge moyen
- 2 j 4 h
- PR mergées (30 j)
- 132
Description
⚙ Compilation target
ES2023
⚙ Library
lib.es5.d.ts
Missing / Incorrect Definition
Current signatures of Math.max and Math.min
max(...values: number[]): number;
min(...values: number[]): number;
should actually be
max(...values: readonly number[]): number;
min(...values: readonly number[]): number;
since they do not modify the rest parameter
Sample Code
const numbers: readonly number[] = [1, 2, 3, 4, 5];
const min = Math.min.apply(undefined, numbers);
Documentation Link
No response
Guide de contribution
Ouvrir le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Piste de recherche
Commencez dans lib.es5.d.ts et examinez les déclarations de Math.min et Math.max. Vérifiez l’exemple readonly number[] par rapport aux signatures actuelles, mettez à jour les définitions afin que l’exemple soit vérifié par le système de types, et confirmez que le comportement existant de Math reste couvert par les tests du projet.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- typescript
- Domaine
- compilers
- Type d'issue
- Bug
- Difficulté
- 1/5
- Temps estimé
- Moins d'une heure
- Activité
- À l'abandon
- Clarté
- Clairement spécifiée
- Accessibilité débutants
- 55/100