-
Notifications
You must be signed in to change notification settings - Fork 8
feat: claude.md 파일 추가 #560
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: claude.md 파일 추가 #560
Conversation
구성 검토 요약워크스루solid-connect-server 프로젝트를 위한 Claude Code 레퍼런스 문서인 CLAUDE.md 파일이 저장소 루트에 새로 추가되었습니다. 다음과 같은 내용들이 포함되어 있습니다:
이 변경사항은 순수하게 문서 기반이며 실행 가능한 로직이나 런타임 변경사항을 포함하지 않습니다. 예상 코드 리뷰 소요 시간🎯 1 (Trivial) | ⏱️ ~3분
Pre-merge checks and finishing touches❌ Failed checks (1 inconclusive)
✅ Passed checks (4 passed)
✨ Finishing touches🧪 Generate unit tests (beta)
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. Comment |
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.
Actionable comments posted: 0
🧹 Nitpick comments (2)
claude.md (2)
56-56: 1. 코드 블록에 언어 지정자 추가마크다운 린팅 규칙에 따라, 몇 개의 코드 블록에서 언어 지정자가 누락되어 있습니다. 다음 위치의 코드 블록들을 수정해주세요:
- Line 56: 트리 구조 → 지정자 없음 (또는
text/plaintext사용)- Line 95: 계층 다이어그램 → 지정자 없음 (또는
plaintext사용)- Line 116: 패키지 구조 → 지정자 없음 (또는
plaintext사용)- Line 223: Fixture 디렉토리 구조 → 지정자 없음 (또는
plaintext사용)- Line 448: Git 커밋 형식 →
text또는plaintext지정자 추가- Line 456: Type 목록 →
text또는plaintext지정자 추가- Line 497: 파일명 형식 → 지정자 없음 (또는
plaintext사용)예:
```plaintext또는 해당하는 언어(예:```java,```bash,```sql)[MD040, fenced-code-language]
Also applies to: 95-95, 116-116, 223-223, 448-448, 456-456, 497-497
603-608: 2. URL을 마크다운 링크 형식으로 변경현재 참고 자료 섹션(Lines 603-608)의 URL들이 문자 그대로 입력되어 있습니다. 마크다운 모범 사례에 따라 다음과 같이 수정해주세요:
- **개발 컨벤션**: [개발-컨벤션-정리](https://github.com/solid-connection/solid-connect-server/wiki/개발-컨벤션-정리) - **테스트 가이드**: `test.md` 파일 참고 - **Spring Boot**: [Spring Boot](https://spring.io/projects/spring-boot) - **JPA**: [Spring Data JPA](https://spring.io/projects/spring-data-jpa) - **TestContainers**: [TestContainers](https://www.testcontainers.org/) - **Flyway**: [Flyway](https://flywaydb.org/)이렇게 하면 마크다운이 더 깔끔하게 렌더링되고, 문서 가독성이 향상됩니다.
[MD034, no-bare-urls]
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
claude.md(1 hunks)
🧰 Additional context used
🪛 markdownlint-cli2 (0.18.1)
claude.md
56-56: Fenced code blocks should have a language specified
(MD040, fenced-code-language)
95-95: Fenced code blocks should have a language specified
(MD040, fenced-code-language)
116-116: Fenced code blocks should have a language specified
(MD040, fenced-code-language)
223-223: Fenced code blocks should have a language specified
(MD040, fenced-code-language)
448-448: Fenced code blocks should have a language specified
(MD040, fenced-code-language)
456-456: Fenced code blocks should have a language specified
(MD040, fenced-code-language)
497-497: Fenced code blocks should have a language specified
(MD040, fenced-code-language)
603-603: Bare URL used
(MD034, no-bare-urls)
605-605: Bare URL used
(MD034, no-bare-urls)
606-606: Bare URL used
(MD034, no-bare-urls)
607-607: Bare URL used
(MD034, no-bare-urls)
608-608: Bare URL used
(MD034, no-bare-urls)
🔇 Additional comments (1)
claude.md (1)
1-619: 3. 문서 전체: 우수한 구조와 완성도전체적으로 이 가이드는 매우 잘 구성되었습니다! 다음 점들이 특히 돋보입니다:
- ✅ 명확한 섹션 구조: 프로젝트 개요부터 주의사항까지 논리적 흐름
- ✅ 실용적 예제: Fixture 패턴, BDD 테스트, 커밋 컨벤션 등 코드 예제 포함
- ✅ 검색 가능한 포맷: 테이블, 코드 블록, 마크다운 형식으로 스캔하기 쉬움
- ✅ 한국어 네이밍 컨벤션 설명: 테스트 가독성 향상을 위한 실용적 가이드
- ✅ 개발자 경험 개선: 마이그레이션 주의사항, 환경별 프로필 등 실무 팁
Claude Code와 개발팀이 이 문서를 통해 빠르고 일관성 있게 기능을 구현할 수 있을 것 같습니다. 위의 두 가지 포맷팅 사항만 해결되면 준비 완료입니다!
Hexeong
left a comment
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~ 좋습니당
관련 이슈
작업 내용
claude.md파일을 추가했습니다! 이거를 앞으로 꾸준히 발전시켜나가면 단순한 기능들은 간편하게 코드를 작성할 수 있을 거 같습니다!
다들 앞으로 열심히 작성해나가봐요!
이 다음 pr로 어드민 지역 관련 curd pr을 올릴건데 이 md파일 기반으로 제가 작성한 거 없이 코드 작성, 테스트코드 작성, 커밋메시지 작성까지 맡겨봤습니다.
아직 좀 손볼 것이 많고 부족한 부분도 보이는데 그래도 확실히 생산성이 높아진 거 같네요!
특이 사항
리뷰 요구사항 (선택)