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

Remove the _babel_filter_ file and skipBabel option from Embroider options #1963

Open
mansona opened this issue Jun 6, 2024 · 1 comment · May be fixed by #1942
Open

Remove the _babel_filter_ file and skipBabel option from Embroider options #1963

mansona opened this issue Jun 6, 2024 · 1 comment · May be fixed by #1942
Assignees
Labels

Comments

@mansona
Copy link
Member

mansona commented Jun 6, 2024

skipBabel is a build option. But in the new way of writing Ember apps, we don't want the build options to control Babel, we prefer the config file babel.config.cjs to be the source of truth. This PR removes skipBabel as a build option and removes the _babel_filter_.js whose content depends on that option.

Issue number #1962 introduces a more standard shape for babel.config.cjs. One piece of this issue is to rely on real babel config options to skip files to achieve what the build option skipBabel used to do. e.g. we could use the https://babeljs.io/docs/options#exclude option 👍 so we don't need to provide a custom escape hatch any more.

That's why the PR for the present issue should be merged only after #1962 is implemented and merged:

@BlueCutOfficial BlueCutOfficial linked a pull request Jun 7, 2024 that will close this issue
@BlueCutOfficial BlueCutOfficial changed the title remove the babel filter file and remove the skipBabel option from embroider options Remove the _babel_filter_ file and skipBabel option from Embroider options Jul 5, 2024
@BlueCutOfficial BlueCutOfficial self-assigned this Jul 5, 2024
@mansona
Copy link
Member Author

mansona commented Jul 11, 2024

This is blocked on #2016

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

Successfully merging a pull request may close this issue.

2 participants