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

feat: init nest monorepo, delete old turborepo config & use pnpm #18

Merged
merged 3 commits into from
Jul 9, 2024

Conversation

0xnigir1
Copy link
Collaborator

@0xnigir1 0xnigir1 commented Jul 8, 2024

🤖 Linear

Closes ZKS-84 ZKS-90

Description

Scaffold the base monorepo using Nest CLI using pnpm. Deletes previous structure from turborepo.
New structure is:

  • apps: contains NestJS applications (the API)
  • libs: will contain library used in apps

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

@0xnigir1 0xnigir1 requested a review from 0xkenj1 July 8, 2024 21:53
Copy link

linear bot commented Jul 8, 2024

ZKS-84 Init monorepo

Use NestJS CLI to init a new monorepo, consider previous zkchainhub repo config

Copy link

linear bot commented Jul 9, 2024

ZKS-90 Create API

Just trigger nest generate app

Copy link
Collaborator

@0xkenj1 0xkenj1 left a 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}’

Copy link
Collaborator

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

Copy link
Collaborator Author

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.
Copy link
Collaborator

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
```
Copy link
Collaborator

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”

Copy link
Collaborator

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

@0xkenj1 0xkenj1 self-requested a review July 9, 2024 19:18
@0xnigir1 0xnigir1 merged commit 26c778c into dev Jul 9, 2024
5 checks passed
@0xnigir1 0xnigir1 deleted the feat/scaffold-nestjs branch July 9, 2024 19:24
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