-
Notifications
You must be signed in to change notification settings - Fork 1
feature: prettier 추가 & eslint와 통합 #9
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
Conversation
Summary of ChangesHello @caseBread, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! 이 PR은 프로젝트에 Prettier를 도입하여 코드 서식의 일관성을 확보하고, 이를 ESLint와 통합하여 개발자들이 정의된 스타일 가이드를 준수하도록 합니다. 이는 코드베이스의 가독성과 유지보수성을 향상시키는 데 기여할 것입니다. Highlights
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. Footnotes
|
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.
Code Review
Prettier를 도입하고 ESLint와 통합하여 코드 스타일을 일관되게 유지하도록 설정한 점이 좋습니다. 전반적으로 변경 사항은 정확하며, 프로젝트의 코드 품질 향상에 기여할 것입니다. 한 가지 개선점으로, Prettier가 의도치 않은 파일을 포맷하지 않도록 .prettierignore 파일을 추가하는 것을 제안합니다. 자세한 내용은 관련 파일에 남긴 코멘트를 참고해주세요.
| "eslint-config-next": "15.5.4", | ||
| "eslint-config-prettier": "^10.1.8", | ||
| "eslint-plugin-prettier": "^5.5.4", | ||
| "prettier": "^3.6.2", |
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.
Prettier를 프로젝트에 추가하신 것을 환영합니다! Prettier가 의도치 않은 파일들(예: 빌드 결과물, node_modules)을 포맷팅하는 것을 방지하기 위해, 프로젝트 루트에 .prettierignore 파일을 추가하는 것을 강력히 권장합니다. ESLint의 ignores 설정은 Prettier CLI를 직접 실행할 때는 적용되지 않기 때문입니다.
다음과 같은 내용으로 .prettierignore 파일을 생성할 수 있습니다:
# Build output
.next/
out/
build/
# Dependencies
node_modules/
# Lock files
pnpm-lock.yaml
package-lock.json
yarn.lock
# Others
next-env.d.ts
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.
p5 논의했던 부분 그대로 잘 들어있는 것 같습니다!
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.
p5 prettier 설치한부분 확인했습니다! 따로 특별한 내용은 없어서 merge 하셔도 될 것 같습니다
|
merge 해도 문제없을 것 같습니다! |
이슈 넘버
구현 사항
📸 스크린샷
Reference
📜 리뷰 규칙
Reviewer는 아래 P5 Rule을 참고하여 리뷰를 진행합니다.
P5 Rule을 통해 Reviewer는 Reviewee에게 리뷰의 의도를 보다 정확히 전달할 수 있습니다.