Skip to content

chore(build): more local tsc build #862

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

Open
wants to merge 9 commits into
base: main
Choose a base branch
from

Conversation

elmerbulthuis
Copy link
Contributor

@elmerbulthuis elmerbulthuis commented Jul 18, 2025

Adds a more localized build setup to the jco package

The jco project now has a tsc script that compiles the project it belongs to. The root project now calls this script.

Also it uses the --build flag wich results in a tsconfig.tsbuildinfo file that will speed up repeated builds by only building the files that changed.

This is a way to a cleaner and easier to understand build architecture. Every project contains everything it needs to build itself, including tasks that can be used to build the project. This PR provides a missing task (tsc) for the jco project. Now we have a way of executing project local tasks in a generic way per environment (in this case, the npm environment).

Then there is a root build system, xtask in our case, that executes those tasks in the right order. In this case there is a root build tasks that compiles all js repositories. It's a pretty simple task now, it just executes the tsc tasks in all projects (1 at the moment). But this could be more complex in the future.

This is in preparation for moving the codebase to TypeScript, see #717.

@elmerbulthuis elmerbulthuis marked this pull request as ready for review July 18, 2025 07:56
@elmerbulthuis elmerbulthuis changed the title More local tsc build chore(build): More local tsc build Jul 19, 2025
@elmerbulthuis elmerbulthuis changed the title chore(build): More local tsc build chore(build): more local tsc build Jul 19, 2025
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.

1 participant