-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
ts: The Great Types Migration #7585
ts: The Great Types Migration #7585
Conversation
Asset Size Report for b09a723 IE11 Builds 🛑 The size of the library EmberData has increased by +6.35 KB (+833.0 B compressed) which exceeds the failure threshold of 75 bytes.Warnings
Changeset
Full Asset Analysis (IE11)
Modern Builds 🛑 The size of the library EmberData has increased by +2.67 KB (+467.0 B compressed) which exceeds the failure threshold of 75 bytes.Warnings
Changeset
Full Asset Analysis (Modern)
Modern Builds (No Rollup) 🛑 The size of the library EmberData has increased by +994.0 B (+161.0 B compressed) which exceeds the failure threshold of 75 bytes.Warnings
Changeset
Full Asset Analysis (Modern)
|
Performance Report for b09a723 Scenario - materialization: ☑️ Performance is stable
Scenario - unload: ☑️ Performance is stable
Scenario - destroy: ☑️ Performance is stable
Scenario - add-children: ☑️ Performance is stable
Scenario - unused-relationships:
|
// and for currentState.stateName to be constructed from flag state. | ||
// Probably just port this work from ember-m3 | ||
// After that we can eliminate this. | ||
if (this.hasRecord && isDSModel(this._record)) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
reminder to discuss with @igorT
types ahoy!
One giant shove over-the-cliff for a library, one-long-review for the man.