microsoft / microsoft/TypeScript
API: incorrect declarations in types.ts
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- Go
- Sterne
- 111k
- Forks
- 14.4k
- Ø Merge
- 1 T. 19 Std.
- Gemergte PRs (30 T.)
- 117
Beschreibung
The --strictNullChecks PR changed some declarations in types.ts to avoid frequent assertions in the compiler's code. Unfortunately these changes make it really unsafe for API users because they no longer contain the actually nullable types: https://github.com/Microsoft/TypeScript/pull/22088#discussion_r184149465 https://github.com/Microsoft/TypeScript/pull/22088#discussion_r184155027
ts.Node#parentis no longer optional (which is definitely wrong forSourceFile)ts.Symbol#declarationsandts.Symbol.valueDeclarationare no longer optional, but can still be undefinedts.Type#symbolis no longer optional, but I guess this comment is still up to date:// Symbol associated with type (if any)
These changes should either be reverted or replaced in the published declaration files.
/cc @andy-ms @weswigham
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne mit den Deklarationen in types.ts und prüfe die verknüpften Diskussionen zu den Änderungen an strictNullChecks. Überprüfe die veröffentlichten Deklarationsdateien für ts.Node#parent, ts.Symbol#declarations, ts.Symbol.valueDeclaration und ts.Type#symbol. Als erledigt gilt die Aufgabe, wenn diese Deklarationen die im Issue beschriebenen nullable oder fehlenden Werte korrekt zulassen.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- typescript
- Bereich
- api
- Issue-Typ
- Bug
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 35/100