Skip to content

chore: [LSG] README.md 이미지 업로드#103

Merged
jaehoon0321 merged 1 commit into
developfrom
dev/LSG
Apr 30, 2026
Merged

chore: [LSG] README.md 이미지 업로드#103
jaehoon0321 merged 1 commit into
developfrom
dev/LSG

Conversation

@LeeSG-0114
Copy link
Copy Markdown
Contributor

💡 PR 제목

예: feat: [KHG] 로그인 API 구현


🔗 관련 이슈

진행 중이면 Refs #123, 완전 해결이면 Closes #123 (완전히 끝났을 때만)


📌 작업 내용 요약

  • 주요 변경 1
  • 주요 변경 2
  • 주요 변경 3

🧱 변경 범위 (Scope)

Backend

  • API 추가/수정
  • Validation/예외처리
  • 인증/인가
  • DB 스키마/쿼리
  • 기타:

Frontend

  • UI/라우팅
  • 상태관리/비동기 로직
  • 입력값/검증
  • 기타:

🧪 테스트 내역

Backend

  • 로컬에서 애플리케이션 실행 확인
  • 단위 테스트 통과
  • API 수동 테스트 (Postman / Swagger 등)

Frontend

  • npm run dev 로컬 실행 확인
  • 주요 화면/기능 수동 테스트

테스트 상세(필수)

테스트한 API/페이지/케이스를 간단히 적어주세요.

  • 예) 로그인 성공/실패, 회원가입, 목록 조회 등

🖼️ 화면 캡처 / 시연 영상 (Frontend 변경 시 권장)

  • before:
  • after:

⚠️ 영향도 / 리스크 / 롤백

리뷰어가 “머지해도 되는지” 판단하기 위한 섹션입니다.

  • 영향도: (예: 기존 API 응답 변경 없음 / 있음)
  • 리스크: (예: 권한 로직 수정, 쿼리 성능, 마이그레이션 필요 등)
  • 롤백 방법: (예: revert 커밋 / 기능 플래그 off / 마이그레이션 롤백 불가 등)

✅ 리뷰어 체크 포인트 (선택)

리뷰어가 집중해서 봐줬으면 하는 부분이 있으면 적어주세요.

  • 예) 트랜잭션 범위, 권한 체크, N+1, 상태관리 로직 등

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request updates the README.md to include a dedicated section for JaCoCo test coverage, documenting the improvement from 53% to 79% coverage. The section was also moved to a more prominent position in the document. Feedback was provided to fix a broken Markdown anchor link caused by uppercase characters and to remove unnecessary trailing whitespace at the end of the file.

Comment thread README.md
Comment thread README.md
@jaehoon0321 jaehoon0321 merged commit 327370e into develop Apr 30, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[chore] README.md 이미지 업로드

3 participants