-
-
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
[FEAT] Strip resolved Deprecations #6814
Conversation
47c18d6
to
a354cfd
Compare
b90341f
to
e4c02d7
Compare
Asset Size Report for 33ba6d5 IE11 Builds EmberData shrank by 955.0 B (167.0 B compressed)If any packages had changed sizes they would be listed here. Changeset
Full Asset Analysis (IE11)
Modern Builds EmberData shrank by 912.0 B (178.0 B compressed)If any packages had changed sizes they would be listed here. Changeset
Full Asset Analysis (Modern)
Modern Builds (No Rollup) EmberData shrank by 910.0 B (175.0 B compressed)If any packages had changed sizes they would be listed here. Changeset
Full Asset Analysis (Modern)
|
058f4e3
to
5adecfe
Compare
ab30a02
to
dbdf3ac
Compare
dbdf3ac
to
ae654fc
Compare
Performance Report for 33ba6d5 Relationship Analysis
|
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.
We may need to test our infra a bit more.
RE tests, your suggestion of writing a test that asserts roughly:
looks great. We should find a way to implement this and then test several different flag levels (LTS 3.8 and 3.12 and then "everything" seem like good candidates). I'll open an issue to track that. |
I also think we need to add documentation (somewhere) explaining how you opt in to this, what it does, etc. |
@rwjblue |
Allows build-time stripping of resolved deprecations from production builds.
For instance, if a deprecation was introduced in 3.13, and the app specifies 3.13 as its minimum version compatibility, any deprecations introduced before or during 3.13 would be stripped away.
This configuration is done by an app's
ember-cli-build
file and looks like this:This PR additionally adds a github-actions test scenario in which we run our tests with all deprecations stripped.
Deprecation flags are named imports from
@ember-data/private-build-infra/deprecations
, e.g.DEPRECATE_{deprecation-id-in-caps}