Skip to content

Conversation

jaejeong1
Copy link
Contributor

답안 제출 문제

체크 리스트

  • PR을 프로젝트에 추가하고 Week를 현재 주차로 설정해주세요.
  • 바로 앞에 PR을 열어주신 분을 코드 검토자로 지정해주세요.
  • 문제를 모두 푸시면 프로젝트에서 Status를 In Review로 설정해주세요.
  • 코드 검토자 1분 이상으로부터 승인을 받으셨다면 PR을 병합해주세요.

@jaejeong1 jaejeong1 self-assigned this Aug 30, 2024
@jaejeong1 jaejeong1 requested a review from a team as a code owner August 30, 2024 15:07
@github-actions github-actions bot added the java label Aug 30, 2024
@jaejeong1 jaejeong1 requested a review from gitsunmin August 30, 2024 15:08
@DaleSeo
Copy link
Member

DaleSeo commented Aug 30, 2024

rebase를 안 하시고 merge를 하셔서, 현재 브랜치가 7 커밋 ahead, 467 커밋 behind 상태네요. 깨끗한 커밋 히스토리 관리를 위해서 다음 PR 부터는 가급적 rebase를 부탁드리겠습니다. 제일 좋은 방법은 미리 remote branch와 fork branch의 sync를 맞추는 거겠죠?

Shot 2024-08-30 at 12 19 11@2x

Shot 2024-08-30 at 12 19 40@2x

이 브랜치에는 최근에 추가된 체크들이 누락되어 1개의 체크만 보입니다. 다른 분들 PR을 보시면 아래와 같이 3개의 체크가 보일 것입니다.

Shot 2024-08-30 at 12 26 06@2x

Copy link
Member

@DaleSeo DaleSeo left a comment

Choose a reason for hiding this comment

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

수고 하셨습니다. 잘 푸셨네요!

@jaejeong1 jaejeong1 merged commit b36b5ea into DaleStudy:main Sep 1, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
No open projects
Status: Completed
Development

Successfully merging this pull request may close these issues.

4 participants