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: 메서드명과 로직의 의미를 통일한다 #102

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

dongho108
Copy link
Collaborator

closed

어떤 기능을 개발했나요?

어떻게 해결했나요?

어떤 부분에 집중하여 리뷰해야 할까요?

(option) 이 부분은 주의해 주세요.

(option) 참고자료

(option) 결과


RCA rule

  • r: 꼭 반영해 주세요. 적극적으로 고려해 주세요.
  • c: 웬만하면 반영해 주세요.
  • a: 반영해도 좋고 안 해도 좋습니다. 사소한 의견입니다.
  • 규칙
    • submit 할 코멘트들 중에서 1개라도 r이 포함되어 있다면 request change를 날린다.
    • r 이 하나도 없다면 approve를 한다.

@dongho108 dongho108 requested a review from ssssujini99 August 27, 2023 23:51
@dongho108 dongho108 self-assigned this Aug 27, 2023
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.

2 participants