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

Improve types for some array internals #20086

Merged
merged 1 commit into from
May 12, 2022

Conversation

wagenet
Copy link
Member

@wagenet wagenet commented May 6, 2022

It's hard to tell in this diff, but some of the EmberArray types changed a little bit to be more accurate.

@wagenet wagenet requested a review from chriskrycho May 6, 2022 19:50
@wagenet wagenet force-pushed the more-internals-ts3 branch from f8a90a1 to 8cf7538 Compare May 6, 2022 20:27
@wagenet wagenet force-pushed the more-internals-ts3 branch from 8cf7538 to e63aed9 Compare May 6, 2022 22:44
@chriskrycho chriskrycho added the TypeScript Work on Ember’s types label May 10, 2022
@wagenet wagenet force-pushed the more-internals-ts3 branch from e63aed9 to af48a98 Compare May 11, 2022 16:31
Copy link
Contributor

@chriskrycho chriskrycho left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Finished a review pass!

@wagenet wagenet force-pushed the more-internals-ts3 branch from af48a98 to 8c6a159 Compare May 12, 2022 17:56
@wagenet wagenet merged commit 6c287c4 into emberjs:master May 12, 2022
@wagenet wagenet deleted the more-internals-ts3 branch May 12, 2022 19:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
TypeScript Work on Ember’s types
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants