microsoft / microsoft/TypeScript

Add readonly modifier to Math.min and Math.max

Abierto
#62,413 5 comentarios 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Bug Domain: check: Type Inference
Lenguaje dominante
Go
Estrellas
111k
Forks
14.3k
Merge medio
2 d 4 h
PR fusionados (30 d)
132

Descripción

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

Guía de contribución

Abrir la guía de contribución

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Línea de trabajo

Comienza en lib.es5.d.ts e inspecciona las declaraciones de Math.min y Math.max. Verifica el ejemplo de readonly number[] con las firmas actuales, actualiza las definiciones para que el ejemplo pase la comprobación de tipos y confirma que el comportamiento existente de Math siga cubierto por las pruebas del proyecto.

Escrito por el modelo de indexación a partir del texto del issue.

Evaluación

Stack tecnológico
typescript
Área
compilers
Tipo de issue
Error
Dificultad
1/5
Tiempo estimado
Menos de una hora
Estado de actividad
Estancado
Claridad
Bien especificado
Aptitud para principiantes
55/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.