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

tracking: replace @ember/debug and enable production asserts #9066

Open
runspired opened this issue Oct 27, 2023 · 0 comments
Open

tracking: replace @ember/debug and enable production asserts #9066

runspired opened this issue Oct 27, 2023 · 0 comments

Comments

@runspired
Copy link
Contributor

Over time many issues have been reported that were due to emergent unspecified behaviors that occur once system expectations have not been met.

Some examples:

Generally speaking we don't want the default to be that asserts are left in production environments, as these massively balloon the size of the project and create performance bottlenecks. But we also want to continue using asserts as inline-unit-tests and guardrails for debug builds.

We have the infra in place though to move to a pattern that would allow some or all asserts to be left in production builds if thats what the user wanted in order to provide better debugging of their app.

Note
Likely we would want to at least specify some asserts as still being debug-only due to their overhead.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Needs Planning
Development

Successfully merging a pull request may close this issue.

1 participant