microsoft / microsoft/TypeScript
Add readonly for all possible function parameter for lib.dom.d.ts
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- Go
- Estrellas
- 111k
- Forks
- 14.4k
- Merge medio
- 1 d 19 h
- PR fusionados (30 d)
- 117
Descripción
Search Terms
library, dom
Suggestion
Add readonly for all possible function parameter for lib.dom.d.ts
Examples
Here is an example for add readonly props for constructor:
declare var URLSearchParams: {
prototype: URLSearchParams;
new(init?: string[][] | Record<string, string> | string | URLSearchParams): URLSearchParams;
toString(): string;
};
It should be
declare var URLSearchParams: {
prototype: URLSearchParams;
new(init?: ReadonlyArray<readonly [string,string]> | Readonly<Record<string, string>> | string | URLSearchParams): URLSearchParams;
toString(): string;
};
then I can use
const myGlobalConfig = {
myURLSearchParamsInit1: [['foo','bar']],
myURLSearchParamsInit2: {foo:'bar'}
} as const;
const params = new URLSearchParams(myGlobalConfig.myURLSearchParamsInit1)
Guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Línea de trabajo
Comienza leyendo la declaración de URLSearchParams en lib.dom.d.ts y compara los tipos de sus parámetros del constructor con el ejemplo readonly del issue. Determina qué otros parámetros de funciones son aptos y hasta qué punto debería aplicarse el cambio. Se considera terminado cuando las declaraciones relevantes de lib.dom.d.ts aceptan de forma coherente entradas readonly sin cambiar su API prevista.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- typescript
- Área
- compilers
- Tipo de issue
- Nueva funcionalidad
- Dificultad
- 5/5
- Tiempo estimado
- Más de una semana
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 25/100