microsoft / microsoft/TypeScript

Add readonly modifier to Math.min and Math.max

Aperta
#62,413 5 commenti 0 reazioni 0 assegnatari Vedi su GitHub
Bug Domain: check: Type Inference
Lingua principale
Go
Stelle
111k
Fork
14.3k
Merge medio
2g 4h
PR unite (30g)
132

Descrizione

### ⚙ 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

```TypeScript
const numbers: readonly number[] = [1, 2, 3, 4, 5];
const min = Math.min.apply(undefined, numbers);
```

### Documentation Link

_No response_

Guida per i contributori

Apri la guida per i contributori

Direzione di ricerca

Inizia in lib.es5.d.ts ed esamina le dichiarazioni di Math.min e Math.max. Verifica l’esempio readonly number[] rispetto alle firme attuali, aggiorna le definizioni in modo che l’esempio superi il controllo dei tipi e conferma che il comportamento esistente di Math continui a essere coperto dai test del progetto.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Stack tecnologico
typescript
Ambito
compilers
Tipo di issue
Bug
Difficoltà
1/5
Tempo stimato
Meno di un'ora
Stato di attività
Ferma
Chiarezza
Specificata chiaramente
Idoneità per principianti
55/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.