microsoft / microsoft/TypeScript
Add readonly for all possible function parameter for lib.dom.d.ts
Nessuno ha ancora preso questa issue.
- Lingua principale
- Go
- Stelle
- 111k
- Fork
- 14.3k
- Merge medio
- 2g 4h
- PR unite (30g)
- 132
Descrizione
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)
Guida per i contributori
Apri la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Direzione di ricerca
Inizia leggendo la dichiarazione di URLSearchParams in lib.dom.d.ts e confronta i tipi dei parametri del suo costruttore con l'esempio readonly nell'issue. Determina quali altri parametri di funzione sono idonei e quanto ampiamente dovrebbe essere applicata la modifica. Il lavoro è completo quando le dichiarazioni pertinenti di lib.dom.d.ts accettano coerentemente input readonly senza modificare la loro API prevista.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- typescript
- Ambito
- compilers
- Tipo di issue
- Funzionalità
- Difficoltà
- 5/5
- Tempo stimato
- Più di una settimana
- Stato di attività
- Ferma
- Chiarezza
- Abbastanza chiara
- Idoneità per principianti
- 25/100