Skip to content

Conversation

@khwww
Copy link

@khwww khwww commented Jan 20, 2026

작업 내용

  • .vscode/settings.json에서 prettier.printWidth: 100 제거
  • prettier.requireConfig: true 추가하여 .prettierrc.json 설정만 따르도록 설정
  • Prettier 포맷팅 오류가 있던 버튼 컴포넌트 수정 (BlockBtn, RoundBtn)

리뷰 요구사항 (선택)

Prettier 설정 통일 방식에 대해 더 좋은 방법이 있다면 이야기 나눠봐도 좋을 것 같아요!

@khwww khwww requested a review from wibaek as a code owner January 20, 2026 16:13
@khwww khwww requested review from enunsnv and manNomi as code owners January 20, 2026 16:13
@vercel
Copy link

vercel bot commented Jan 20, 2026

@khwww is attempting to deploy a commit to the solid-connection Team on Vercel.

A member of the Team first needs to authorize it.

@coderabbitai
Copy link

coderabbitai bot commented Jan 20, 2026

Walkthrough

이 변경사항은 세 개의 파일에 걸쳐 포매팅 및 설정 조정을 포함합니다.

  1. Prettier 설정 업데이트: .vscode/settings.json 파일에서 prettier.printWidth 속성이 제거되고 prettier.requireConfig 속성이 true로 설정되었습니다.

    • 이제 Prettier는 프로젝트의 설정 파일을 필수적으로 참조하게 됩니다.
  2. 컴포넌트 인터페이스 포매팅: BlockBtn.tsxRoundBtn.tsx의 인터페이스 선언에서 extends 절이 줄바꿈으로 분할되었습니다.

    • 타입 정의나 동작에는 변화가 없으며, 코드의 가독성을 위한 라인 너비 조정입니다.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~4 minutes

Suggested reviewers

  • wibaek
  • enunsnv
🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed PR 제목이 변경사항의 주요 목적을 명확하게 요약하고 있으며, Prettier 설정 통일과 포맷팅 수정이라는 핵심 내용을 잘 전달하고 있습니다.
Description check ✅ Passed PR 설명에서 필수 섹션인 '작업 내용'은 완전히 작성되었으나, '관련 이슈' 섹션이 누락되어 있습니다.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Fix all issues with AI agents
In @.vscode/settings.json:
- Line 2: You added "prettier.requireConfig": true to .vscode/settings.json but
the repo lacks a .prettierrc.json; create a .prettierrc.json at the repository
root and populate it with the required Prettier settings (e.g., tabWidth/indent,
semi, singleQuote, printWidth, trailingComma) so Prettier runs locally when
"prettier.requireConfig" is enabled; ensure the new file reflects your project's
conventions and is committed to the repo.

Copy link
Contributor

@manNomi manNomi left a comment

Choose a reason for hiding this comment

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

고생하셨습니다!

/)__/)
( • 0 • ) 🥕
^^

@@ -1,5 +1,5 @@
{
"prettier.printWidth": 100,
"prettier.requireConfig": true,
Copy link
Contributor

Choose a reason for hiding this comment

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

혹시 이값 true로 수정하신 이유가 있을까요 ?

Copy link
Author

@khwww khwww Jan 20, 2026

Choose a reason for hiding this comment

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

개인의 전역 설정이 prettier에 적용되는 경우를 방지하기 위해서
.prettier.json 설정을 기준으로 일관되게 포맷되도록 했습니다!
(저게 충돌원인은 아닌 것 같아요)

Copy link
Contributor

Choose a reason for hiding this comment

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

오 좋네요~!!
아마 프리티어.json이 있어서 해당 옵션이 쓰일일은 없겠지만 명시적으로 개인설정이 적용되는걸 막으면 좋겠네요

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants