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

output macros-config.json and _babel_config_ in the .embroider folder #1935

Merged
merged 2 commits into from
May 22, 2024

Conversation

BlueCutOfficial
Copy link
Collaborator

@BlueCutOfficial BlueCutOfficial commented May 22, 2024

Context

We are still working on removing the rewritten app completely. To achieve this goal, we need to remove all the remaining differences between the initial Ember app and the rewritten app generated at stage 2.

One of these differences is the presence of macros-config.json and _babel_config_.js in the root folder of the rewritten-app.

Solution

Instead of outputting macros-config.json and _babel_config_.js in the rewritten app, we now output them directly in the .embroider folder, which contains all the config files we need.

…er folder so it's separated from the rewritten-app
@BlueCutOfficial BlueCutOfficial marked this pull request as ready for review May 22, 2024 10:10
@BlueCutOfficial BlueCutOfficial marked this pull request as draft May 22, 2024 10:10
@BlueCutOfficial BlueCutOfficial force-pushed the align-package-json-mcros-config branch 7 times, most recently from 7295b17 to e970959 Compare May 22, 2024 13:13
…r folder so it's separated from the rewritten-app
@BlueCutOfficial BlueCutOfficial force-pushed the align-package-json-mcros-config branch from e970959 to 187f072 Compare May 22, 2024 13:29
@BlueCutOfficial BlueCutOfficial marked this pull request as ready for review May 22, 2024 14:10
@mansona mansona changed the title Feat(remove rewritten-app) / output macros-config.json in the .embroider folder output macros-config.json and _babel_config_ in the .embroider folder May 22, 2024
@mansona mansona added the enhancement New feature or request label May 22, 2024
@mansona mansona merged commit 93b39c4 into main May 22, 2024
176 of 177 checks passed
@mansona mansona deleted the align-package-json-mcros-config branch May 22, 2024 14:11
@github-actions github-actions bot mentioned this pull request May 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants