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

[Experimental] Use yarn.build to build and test the framework #3851

Closed
wants to merge 7 commits into from

Conversation

jtoar
Copy link
Contributor

@jtoar jtoar commented Dec 5, 2021

This PR uses yarn.build to build and test the framework. This is in place of lerna.

Note that this PR doesn't totally rip out lerna as we still use it for publishing.

yarn build and test are yarn 3 plugins. They run the build and test scripts defined in the package.json of each package in the workspace. They understand a package's dependency graph, and also have a built-in caching mechanism that avoids rebuilding/retesting a package if it hasn't been modified since since the last build/test (though this can be easily overruled via --ignore-cache).

-> build

yarn.build--build.mp4

-> test

yarn.build--test.mp4

Although merging this PR would mean that yarn build and yarn test always refers to the plugin, if the original build/test scripts are ever needed, they're still accessible via yarn run build and yarn run test.

Questions:

  • I haven't added back build:watch yet (at the framework level; each package has it though). does yarn build have a built-in watching feature?
  • how does the caching mechanism actually work?

Also see:

@jtoar
Copy link
Contributor Author

jtoar commented May 5, 2022

Will try to revisit this at some point, but closing for now as there are other higher-priority yarn-related improvements that could be made.

@jtoar jtoar closed this May 5, 2022
@jtoar jtoar deleted the ds-try-yarn.build branch October 5, 2022 00:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Archived
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants