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

chore: no builds #3074

Open
wants to merge 25 commits into
base: rc
Choose a base branch
from
Open

chore: no builds #3074

wants to merge 25 commits into from

Conversation

tracy-french
Copy link
Contributor

@tracy-french tracy-french commented Nov 27, 2024

Overview

This change removes the need to build dependent packages when developing to incorporate their changes. During development, the TypeScript code between packages is used directly. Changes in one package are immediately reflected in other packages. Development tasks, including the storybook dev server, unit tests, and linting, as well as editor squiglies, will see immediate effect of changing code across packages without changes. The dev env has a strong HMR effect with no page refresh. Watch mode unit tests automatically re-run. It's awesome!

Taking advantage of this change, this PR also includes:

  • Definition of root npm start script set to start the development environment (dashboard only currently but will be updated).
  • Split build, lint, and unit test jobs and further shard playwright and unit tests to reduce cache miss GH action time to ~5 minutes max.

Legal

This project is available under the Apache 2.0 License.

@tracy-french tracy-french force-pushed the chore-no-build branch 9 times, most recently from 0f1490c to c9bb02c Compare November 27, 2024 21:40
@tracy-french tracy-french changed the title chore: remove need to build packages chore: no builds Nov 27, 2024
@tracy-french tracy-french marked this pull request as ready for review November 27, 2024 21:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants