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

DEV: Patch deprecated-run-loop-and-computed-dot-access in production #25074

Merged
merged 1 commit into from Jan 2, 2024

Conversation

davidtaylorhq
Copy link
Member

@davidtaylorhq davidtaylorhq commented Dec 29, 2023

In Ember, these deprecations are wrapped in an if(DEBUG) check, so they are optimized out of the production build. We prefer to keep deprecations in production so that we can collect telemetry and warn theme authors who do not use local development environments.

This commit restores the deprecations as part of our ember-production-deprecations addon.

These will be removed once we finalise the upgrade to Ember 5.

In Ember, these deprecations are wrapped in an `if(DEBUG)` check, so they are optimized out of the production build. We prefer to keep deprecations in production so that we can collect telemetry and warn theme authors who do not use local development environments.

This commit restores the deprecations as part of our ember-production-deprecations addon.
@discoursebot
Copy link

This pull request has been mentioned on Discourse Meta. There might be relevant details there:

https://meta.discourse.org/t/topic-list-previews-theme-component/209973/398

@davidtaylorhq davidtaylorhq merged commit 641c689 into main Jan 2, 2024
19 checks passed
@davidtaylorhq davidtaylorhq deleted the runloop-computed-dot-access branch January 2, 2024 10:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
3 participants