xeokit / sdk

Next-generation alpha-status xeokit viewer in development
https://xeokit.github.io/sdk/docs/api/index.html
GNU Affero General Public License v3.0
18 stars 10 forks source link

Feat/monorepo reconfiguration #12

Closed DhivinX closed 6 months ago

DhivinX commented 6 months ago

From now, building packages should work without any problems, just use pnpm dist. (Before doing so, remove all node_modules generated by yarn/npm) See README.md in section Project development.

Changes introduced by this PR:

Planning:

We could add Github Actions configuration to automate the publishing of packages when changes are pushed to the main branch. You would need to add a develop branch for the current code and a main branch would be the alive branch. Then when we merge the develop branch into main it would automatically publish the packages to the registry.

We could also add a dependabot configuration to keep external dependencies up to date.

xeolabs commented 6 months ago

Looks good to me, thanks very much @DhivinX

paireks commented 6 months ago

Wow, I can finally succesfully build it! :)

The only thing I had to do additionally is that between "npm i pnpm -g" and "pnpm install turbo --global" I had to restart the computer, because of that thing that happens on Windows and Macs: https://stackoverflow.com/questions/77224607/how-can-i-help-pnpm-find-its-global-directory

The last thing I personally miss right now probably is running the unit tests existing. @DhivinX have you tried running all of them, or some of them? If so: can the process be written down somewhere, so I can give it a try?

DhivinX commented 6 months ago

@paireks I will try to fix the tests in the next iteration.