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

Monorepo #466

Merged
merged 28 commits into from
Dec 5, 2024
Merged

Monorepo #466

merged 28 commits into from
Dec 5, 2024

Conversation

funwithtriangles
Copy link
Member

@funwithtriangles funwithtriangles commented Nov 4, 2024

I've been to config hell and back! We now have a monorepo, with three packages:

  • @hedron/engine (will eventually be on npm)
  • @hedron/desktop (depends on @hedron/engine)
  • @hedron/example-project (still all sorts of type errors here but we can worry about that later)

For some boring technical reason we're back to using yarn. You can do everything from the root of the project, see the readme file. It's important that @hedron/engine is built before you start developing, otherwise you'll see a bunch of errors. yarn dev should do everything for you.

Let me know if this is working for you @cale-bradbury. I'm also fully aware how much this is going to mess any PRs you might have coming in. Not sure what's going to be easier to merge first, although I feel like there's less chance of changes getting lost if we merge this first and then you'll have to manually move your changed files into this new system.

Last fixes

  • add dependencies to engine package.json
  • remove any drops from desktop package.json

@funwithtriangles funwithtriangles changed the base branch from dev to alpha November 4, 2024 15:32
@funwithtriangles funwithtriangles merged commit fcc8888 into alpha Dec 5, 2024
1 check passed
@funwithtriangles funwithtriangles deleted the monorepo branch December 5, 2024 12:49
@cale-bradbury
Copy link
Contributor

sorry for the late reply, it's working on windows after a yarn / yarn dev

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.

2 participants