-
Notifications
You must be signed in to change notification settings - Fork 0
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
feat: init nest monorepo, delete old turborepo config & use pnpm #18
Conversation
ZKS-84 Init monorepo
Use NestJS CLI to init a new monorepo, consider previous zkchainhub repo config |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Gj ser!! Can we check which is the way that NestJS handles lib imports within apps ?
I think Nest uses aliases to resolve paths. Lets make sure that we import libs with ‘@zkchainhub/{libname}’
.vscode/settings.json
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not sure if we need this file, if we don’t lets remove it
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we need it so we all have the same 'formatOnSave' configuration for vscode and share the same extension for ESLint
README.md
Outdated
@@ -4,59 +4,40 @@ | |||
|
|||
The zkChainHub project for zkSync involves the creation of a comprehensive data management system. This project is designed to provide key insights into the zkSync network, making it an invaluable tool for users and developers. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should add a better description for the project, probably the tech design doc would provide some help
README.md
Outdated
``` | ||
```bash | ||
$ nest g library my-library | ||
``` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lets also add “Creating a new app”
lint-staged.config.js
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lets see if we can replace this with some line in package.json , to avoid adding a file with just one line
🤖 Linear
Closes ZKS-84 ZKS-90
Description
Scaffold the base monorepo using Nest CLI using pnpm. Deletes previous structure from turborepo.
New structure is:
Note: didn't delete packages folder so later we have it there to refactor into a library but is not part of new src code