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

Improve "sync:*" commands #253

Open
IlCallo opened this issue Jun 1, 2022 · 0 comments
Open

Improve "sync:*" commands #253

IlCallo opened this issue Jun 1, 2022 · 0 comments
Labels
enhancement New feature or request

Comments

@IlCallo
Copy link
Member

IlCallo commented Jun 1, 2022

Those commands are available into test projects and take care of building one of the AEs in the monorepo and installing it into the test-project, to quickly validate if the changes we did are actually working

Right now they're copy/pasted in every test-project and not really efficient

We should create a JS script taking in a few arguments, instead of relying on copy/paste, as their structure is always the same

Additionally, they rebuild each AE for every test project they're executed in, which right now results in a total of 9 rebuilds if you want to test all projects in a single run, but we actually only needed to re-build AEs one time per AE when running a possible global "sync:all" command

It would be even better to avoid rebuilding an AE if no source files changed in that AE and the dist folder is already available with the latest version. Not sure how to perform this check tho

@IlCallo IlCallo added the enhancement New feature or request label Jun 1, 2022
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

No branches or pull requests

1 participant