Skip to content
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

[Refactor/#15] 프로젝트 이름 변경 및 CheckStyle 업데이트 #17

Merged
merged 5 commits into from
Feb 6, 2024

Conversation

Starlight258
Copy link
Collaborator

@Starlight258 Starlight258 commented Feb 6, 2024

🔗 Linked Issue

🛠 개발 기능

  • 프로젝트 이름을 서비스명으로 변경
  • CheckStyle에서 Javadoc 경고 뜨지않도록 변경

🧩 해결 방법

  • 프로젝트명 변경시 build.gradle 파일과 application 실행 객체, test 디렉토리 부분까지 모두 변경한다.
  • CheckStyle 파일에서 Javadoc 부분을 주석처리한다.

🔍 리뷰 포인트

  • checkstyle을 github actions에 포함시키려다가 에러가 나서 우선은 주석 처리했습니다 !


📋 Code Review Priority Guideline

  • 🚨 P1: Request Change
    • 필수 반영: 꼭 반영해주시고, 적극적으로 고려해주세요 (수용 혹은 토론).
  • 💬 P2: Comment
    • 권장 반영: 웬만하면 반영해주세요.
  • 👍 P3: Approve
    • 선택 반영: 반영해도 좋고 넘어가도 좋습니다. 그냥 사소한 의견입니다.

Copy link
Collaborator

@FacerAin FacerAin left a comment

Choose a reason for hiding this comment

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

빠른 이슈 해결 감사합니다 👍👍
checkStyle action도 천천히 확인 부탁드려요! 😄

@Starlight258 Starlight258 merged commit 8b9ad6b into dev Feb 6, 2024
1 check passed
@Starlight258 Starlight258 deleted the feat/update-project-name-and-style branch February 6, 2024 08:40
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