Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update to TS v4.9 #20285

Merged
merged 1 commit into from
Nov 29, 2022
Merged

Update to TS v4.9 #20285

merged 1 commit into from
Nov 29, 2022

Conversation

chriskrycho
Copy link
Contributor

@chriskrycho chriskrycho commented Nov 29, 2022

This lets us take advantage of much smarter in-based type narrowing and the new satisfies operator internally. We'll need to make a decision about what versions to support when we actually "stabilize" the types: not merely publishing them, but opting consumers into them by default. However, it's likely that 4.9 and 5.0 will be reasonable targets at that point, and even if not the particular 4.9 features we might rely on are "runtime" TS features, so we should be able to add a CI check for 4.8 if we want to add support for it in the test matrix.

@chriskrycho chriskrycho added internal TypeScript Work on Ember’s types labels Nov 29, 2022
@chriskrycho chriskrycho disabled auto-merge November 29, 2022 15:03
@chriskrycho chriskrycho force-pushed the add-ts-4.9-explicitly branch from 971e443 to e00d427 Compare November 29, 2022 15:05
@chriskrycho chriskrycho changed the title Explicitly add TS v4.9 to test matrix Update to TS v4.9 Nov 29, 2022
This lets us take advantage of much smarter `in`-based type narrowing
and the new `satisfies` operator internally. We'll need to make a
decision about what versions to support when we actually "stabilize"
the types: not merely publishing them, but opting consumers into them
by default. However, it's likely that 4.9 and 5.0 will be reasonable
targets at that point, and even if not the particular 4.9 features we
might rely on are "runtime" TS features, so we should be able to add a
CI check for 4.8 if we want to add support for it in the test matrix.
@chriskrycho chriskrycho force-pushed the add-ts-4.9-explicitly branch from e00d427 to 1c779a4 Compare November 29, 2022 15:14
@chriskrycho chriskrycho merged commit 233c702 into master Nov 29, 2022
@chriskrycho chriskrycho deleted the add-ts-4.9-explicitly branch November 29, 2022 15:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
internal TypeScript Work on Ember’s types
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant