-
-
Notifications
You must be signed in to change notification settings - Fork 3
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
build: prepare monorepo #35
Comments
Hey! I'm interested in contributing to create the vue version of beautiful tree. How can I get started? |
Hi @bhishekprajapati 😄 , If you feel brave enough to pick the monorepo task first (as it's kind of a precondition), you can start with it. If you don't feel comfortable with the monorepo task, I can prioritise it on my side, so you can start with Vue after we have it in place (As I was working on it alone, I wasn't prioritising it yet). NOTE: you can pick any task and leave a message so we can know that you are on it.
Once you have your changes, you can send them as a PR and I'll review them. |
Hi @castarco , |
Hey @castarco, do you any discord server or slack? |
Hi @bhishekprajapati , I'm sorry for the delay, I'm having a busy week. For the time being, there are no "real time" channels, but I'll prepare something this weekend 👍🏻 . |
@castarco |
Hi @bhishekprajapati .
|
Going back to the main topic of this issue, I started working on it. |
Fixes #35 In the long term, this monorepo structure won't be enough. We'll have to add some extra automations, but for now it's good to go, as it allows us to unblock many other tasks!! 🎉 🚀
Reorganise the code so the package
@beautiful-tree/react
is not at the root level of the repository, and we can have other packages as well.Ideally, it should be done in a way that changes in a package only trigger the CI flow associated to that code and anything that depends on it, but nothing else (so we don't slow down our CI pipelines with useless work). Some tools that could help are Turbo Repo, or Nx.
This one is a precondition for the following tasks:
The text was updated successfully, but these errors were encountered: