microsoft / microsoft/TypeScript
Bloomberg feedback for 5.8
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- Go
- Sterne
- 111k
- Forks
- 14.3k
- Ø Merge
- 2 T. 4 Std.
- Gemergte PRs (30 T.)
- 132
Beschreibung
Acknowledgement
- I acknowledge that issues using this template may be closed without further explanation at the maintainer's discretion.
Comment
We evaluated the 5.8 beta and RC releases and 5.8 seems to be a low impact release for us.
We are excited to adopt the new flag --erasableSyntaxOnly for the improved Editor experience for features we already avoid. A significant number of our projects already comply with the corresponding rules due to custom error checks that will happily become redundant. We remain interested in gaining the ability to use first-class enums that comply with the TS=JS+types model. For example via the erasable as enum syntax proposal.
| # | Change | Affects | Release notes | Packages affected |
|---|---|---|---|---|
| 1 | Computed properties with a non literal type now appear in declarations | Declaration Emit | Yes | >1% |
| 2 | Better type checking of conditional expressions | Type Checking | ~1-2% |
Computed properties with a non literal type now appear in declarations
Some declaration files changed and now preserve computed properties where index signatures would previously be created. This is an expected change and part of ongoing Isolated Declarations work.
// index.ts
export let foo = "x"
export class Bar {
[foo]: string
}
// index.d.ts
export declare let foo: string;
export declare class Bar {
// in 5.8
[foo]: string;
// in 5.7
[x: string]: string;
}
Better checking in conditional expressions
We observed roughly a dozen new compile errors relating to improved conditional return checking.
While not directly announced the change seems related to Checked Returns for Conditional and Indexed Access Types
In TypeScript 5.7 and before, if a branch of a conditional expression returned any we would not get an error if we returned the wrong type on the other branch. Now we do, at least if the expression is in the return of a function:
declare const data: number | string | undefined;
const result = (): number =>
typeof data === 'string'
? null as any
: data; // Error now
const result2: number =
typeof data === 'string'
? null as any
: data; // Still not an error ? 😕
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 damit, das Beispiel für einen bedingten Ausdruck im verknüpften TypeScript Playground mit 5.7.3 und 5.8 zu reproduzieren, und vergleiche anschließend das gemeldete Beispiel für declaration emit zwischen den Versionen. Das Issue dokumentiert beobachtetes Verhalten, nennt jedoch weder eine Quelldatei noch einen Test oder eine spezifische gewünschte Änderung. Daher wäre für den Abschluss eine Anleitung von Maintainer erforderlich, ob eines der beiden Verhaltensweisen fehlerhaft ist.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- typescript
- Bereich
- compilers
- Issue-Typ
- Bug
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Muss geklärt werden
- Anfängerfreundlichkeit
- 25/100