Skip to content

Conversation

minji-go
Copy link
Contributor

@minji-go minji-go commented Dec 8, 2024

답안 제출 문제

체크 리스트

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

@minji-go minji-go requested a review from a team as a code owner December 8, 2024 09:35
@github-actions github-actions bot added the java label Dec 8, 2024
@minji-go minji-go marked this pull request as draft December 8, 2024 09:37
@minji-go minji-go marked this pull request as ready for review December 8, 2024 09:39
@HC-kang
Copy link
Contributor

HC-kang commented Dec 8, 2024

@minji-go 님, 안녕하세요!
주차별 풀이 제출을 위해서는 프로젝트와 현재 주차를 등록해주셔야 해요.
다만 확인해보니, 아직 리트코드 3기 팀에 조인하시지 않은 것으로 보이는데요, 혹시 팀 초대 메일을 받지 못하셨다면 멘션해주세요!

@minji-go minji-go requested a review from gwbaik9717 December 9, 2024 00:32
@HC-kang
Copy link
Contributor

HC-kang commented Dec 9, 2024

@minji-go 민지님, 개행문자 추가 후에도 실패한 케이스가 있으셔서 말씀드려요!
저희 CI 과정에 약간의 버그가 있었어서, 수정 반영하였는데요!
최신 변경사항 반영 후에 다시 체크� 해 주실 수 있을까요?

Copy link
Contributor

@obzva obzva left a comment

Choose a reason for hiding this comment

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

안녕하세요 풀이하시느라 수고하셨습니다 😄
전반적으로 코드가 간결해서 리뷰하기 수월했습니다
아직 시간이 꽤 남았으니 각 풀이에 대해 Big O 분석을 남겨주시면 더 좋을 것 같습니다

@minji-go
Copy link
Contributor Author

@obzva Big O 분석해봤는데 확인 후 승인 부탁드립니다! 🐰

@obzva
Copy link
Contributor

obzva commented Dec 13, 2024

@gwbaik9717 안녕하세요 다른 분의 PR을 리뷰해주는 것까지가 참여자의 역할입니다 리뷰 진행해주세요~
@minji-go 우선적으로는 PR 승인 및 검토 요청을 3기 참여자분께 해주세요~ 코치도 리뷰를 하지만 기본적으로는 참여자끼리 리뷰를 주고받아야 합니다 :)

@minji-go
Copy link
Contributor Author

@obzva 아! 처음이라 제대로 몰랐네요 :-0 리뷰 감사합니다 ㅎㅎ

Copy link
Contributor

@gwbaik9717 gwbaik9717 left a comment

Choose a reason for hiding this comment

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

안녕하세요, @minji-go 님! 리뷰가 좀 늦었습니다. 전반적으로 코드가 깔끔해서 이해하기 쉬웠어요. 몇몇 군데 코멘트 남겨서 참고부탁드리겠습니다 :)

@SamTheKorean SamTheKorean merged commit 4a2b8b5 into DaleStudy:main Dec 15, 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.

5 participants