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.

Sign up for GitHub

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

Jump to bottom

[Feature] Initial project setup #1

Merged
merged 20 commits into from
Nov 23, 2024
Merged

[Feature] Initial project setup #1

merged 20 commits into from
Nov 23, 2024

Conversation

jiaah
Copy link
Collaborator

@jiaah jiaah commented Nov 18, 2024 •
edited
Loading

Purpose

개발을 시작하기 위한 초기 프로젝트 설계와 eslint, prettier, ts 등의 기본 설정을 완료했습니다.

Changes

  • 모노레포 기반의 FSD 구조 설계
  • 기술 스택:
    • Frontend: Next.js, Vanilla Extract, TypeScript
    • Backend: Node.js, Supabase
  • README를 수정

@jiaah jiaah added the good first issue Good for newcomers label Nov 18, 2024
@jiaah jiaah requested a review from f-lab-kevin November 18, 2024 00:48
@jiaah jiaah self-assigned this Nov 18, 2024
f-lab-kevin
f-lab-kevin approved these changes Nov 20, 2024
View reviewed changes
Copy link

@f-lab-kevin f-lab-kevin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

첫 세팅 고생많으셨습니다!

지금 구성한 모노레포 구조에 대한 의존성 그래프를 그림으로 표현해 Readme에 추가해보시죠!

.yarnrc.yml Outdated
@@ -0,0 +1,2 @@
yarnPath: .yarn/releases/yarn-4.5.1.cjs
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pnpm을 이용한 모노레포구조가 아니라 Yarn Workspace를 활용한 방식일까요?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

네!

packages/.DS_Store Outdated
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

불필요한 파일인것 같습니다

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

제거했습니다!

@f-lab-kevin
Copy link

Turborepo나 Nx와 같은 툴들을 이용해보시는것도 고려해보시죠!

@f-lab-kevin
Copy link

Readme에 프로젝트 초기세팅 및 시작과정에 대해서도 작성해주세요

@jiaah
Copy link
Collaborator Author

jiaah commented Nov 20, 2024

첫 세팅 고생많으셨습니다!

지금 구성한 모노레포 구조에 대한 의존성 그래프를 그림으로 표현해 Readme에 추가해보시죠!

추가했어요!

@jiaah
Copy link
Collaborator Author

jiaah commented Nov 20, 2024

Turborepo나 Nx와 같은 툴들을 이용해보시는것도 고려해보시죠!

빌드 캐싱, 병렬 실행 최적화, 의존성 기반 빌드 순서 관리 등의 기능을 제공하는Turborepo를 사용하는게 좋겠어요! 그리고 Turborepo를 사용시에는 pnpm을 사용하는 것이 궁합이 더 좋은 것 같으니 pnpm으로 마이그레이션을?!

@jiaah
Copy link
Collaborator Author

jiaah commented Nov 20, 2024

Readme에 프로젝트 초기세팅 및 시작과정에 대해서도 작성해주세요

README에 #How to run (초기세팅 및 시작과정)이 작성되어 있고,
#Documetation에 초기 세팅 과정에서의 구조적인 고민과 트러블 슈팅을 작성한 링크를 추가해두었어요.

이 외에 어떤 부분을 추가적으로 작성하는게 좋을까요?

@jiaah
Copy link
Collaborator Author

jiaah commented Nov 20, 2024 •
edited
Loading

Turborepo나 Nx와 같은 툴들을 이용해보시는것도 고려해보시죠!

빌드 캐싱, 병렬 실행 최적화, 의존성 기반 빌드 순서 관리 등의 기능을 제공하는Turborepo를 사용하는게 좋겠어요! 그리고 Turborepo를 사용시에는 pnpm을 사용하는 것이 궁합이 더 좋은 것 같으니 pnpm으로 마이그레이션을?!

Yarn에서 pnpm + Turborepo로 마이그레이션 완료했습니다.! (9f90946)

@sonarqubecloud SonarQubeCloud
Copy link

Quality Gate Passed Quality Gate passed

Issues
1 New issue
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@jiaah jiaah merged commit 38b6636 into main Nov 23, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@f-lab-kevin f-lab-kevin f-lab-kevin approved these changes

Assignees

@jiaah jiaah

Labels
good first issue Good for newcomers
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jiaah @f-lab-kevin