-
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
chore(*): 패키지 버전 설정 및, 파일 정리 #3
Conversation
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.
오! 고생하셨습니다 🎉
packages 아래에 있는 패키지명 @repo/
-> @sambad/
로 바꾸는 건 어떨까요!
그냥 지울까 하다가도.. 어차피 쓸 것 같아서 패키지명만 바꿔도 괜찮을 거 같아요!
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.
LGTM~~~ 몇 가지 궁금한 점 리뷰로 남겼습니다. 😇
@@ -1,4 +1,4 @@ | |||
This is a [Next.js](https://nextjs.org) project bootstrapped with [`create-next-app`](https://nextjs.org/docs/app/api-reference/create-next-app). | |||
This is a [Next.js](https://nextjs.org/) project bootstrapped with [`create-next-app`](https://github.com/vercel/next.js/tree/canary/packages/create-next-app). |
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.
혹시 요런 수정사항은 pnpm install 하면서 생긴건가여?? 아니면 직접 수정하신건가요?? 궁금해서 질문드립니다.
:root { | ||
--background: #ffffff; | ||
--foreground: #171717; | ||
--max-width: 1100px; |
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.
모바일 가로 화면 잡으실 때 body max-width는 PC 사이즈로 두시고, 따로 Root Layout Component를 통해서 잡으시나여?
혹시 1100px은 어떤 기준인가요??
🎉 변경 사항
apps 디렉토리 하위의 서비스 디렉토리의 next 버전과 react 버전을 픽스했습니다.
🔗 링크
https://www.notion.so/apps-fc27c354573d40d4b42b78873fb1dc1f?pvs=4
🙏 여기는 꼭 봐주세요!