chore: Switch to turborepo - #128
Conversation
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
|
Warning Review the following alerts detected in dependencies. According to your organization's Security Policy, it is recommended to resolve "Warn" alerts. Learn more about Socket for GitHub.
|
8eaad57 to
1051500
Compare
786d59c to
70b668c
Compare
70b668c to
3ca4008
Compare
|
We currently have doc outlining a plan to do just this - convert to a monorepo and typescript. We are going to approach this as a multi-step change to make it possible to review and confirm everything still works and behaves the exact same way. This is too large of a change for a single PR. |
|
@justinformentin same thoughts - although turbo is just one of many pieces of the puzzle. #125 contains everything needed to get this repo up to speed:
The PR is massive and the idea is to split it up into 10+ I guess steps. But to make it safe, E2E regression test suite is essential. To make things difficult, of course is much harder to add it first, then introduce turbo, but it's doable. Also for browser testing ( |
In this PR
cd xxx && npm ito aturbobased setup that mirrorssdk(in fact, it has been largely if not completely copied from there). This brings a lot of goodies inverifyEnclaveSignaturefunction from tests. Basically the tests were testing that a mocked function works as expected