-
Notifications
You must be signed in to change notification settings - Fork 2
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] React + Vite + Typescript 초기 설정 #97
Conversation
- tsconfig 설정 추가 - eslint path, alias 설정 추가 - vite 설정 추가
This PR exceeds the recommended size of 1000 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size. |
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.
빠른 작업 너무 감사합니다 석호님!!
반드시 해야하는 것들은 아니지만 초기 세팅 PR에서 미리 해두면 좋을 자잘한 작업들에 대해 코멘트 남겨놨습니다! 확인해주시면 감사하겠습니다 🙇🙇
apps/web/.eslintrc.cjs
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.
P1
react 프로젝트를 생성하면서 web이라는 폴더가 만들어진 것 같은데,
web 폴더로 빼지 않고 현재 web 하위에 있는 파일을 전부 client 폴더로 옮겨주시면 어떨까요?
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.
apps
라는 디렉토리에 네이밍에 맞게 web
으로 수정했습니다. 이전으로 되돌리는게 괜찮으면 수정하겠습니다!
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.
아 검색해보니 turborepo로 모노레포를 구성할 때 apps/web이라는 네이밍이 관용적으로 많이 쓰이는 것 같네요! 제가 잘 몰랐습니다.
알아봤을 땐 web이 브라우저 상에서 동작하는 클라이언트를 나타내기 위해 쓰이는 네이밍인 것 같아요.
그렇다면 기존의 client라는 폴더는 지우고 web이라는 폴더로 통합하면 어떨까 해요!
다음은 클로드가 추천해준 폴더 네이밍입니다 ㅎㅎ
apps/
├── web/ # 프론트엔드 메인 앱
├── api/ # 메인 백엔드 API 서버
└── worker/ # 백그라운드 작업 서버
...
packages/ # 공통 모듈
├─config
├─utils
...
apps/web/src/App.tsx
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.
P2
App.tsx도 불필요한 초기 코드 없이 빈 태그로만 남겨주시면 더 좋을 것 같습니다!!
This PR exceeds the recommended size of 1000 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size. |
관련 이슈 번호
작업 내용
packages
에 정의한 설정을 확장하여 설정 진행vite-tsconfig-paths
를 통한 절대 경로 인식public
폴더의 경로를/
으로 설정하여 발생하는 eslint 오류 해결스크린샷
pnpm dev
명령 실행시 서버 실행