Skip to content

Conversation

river20s
Copy link
Contributor

@river20s river20s commented Dec 14, 2024

답안 제출 문제

체크 리스트

  • 우측 메뉴에서 PR을 Projects에 추가해주세요.
  • Projects의 오른쪽 버튼(▼)을 눌러 확장한 뒤, Week를 현재 주차로 설정해주세요.
  • 바로 앞에 PR을 열어주신 분을 코드 검토자로 지정해주세요.
  • 문제를 모두 푸시면 프로젝트에서 StatusIn Review로 설정해주세요.
  • 코드 검토자 1분 이상으로부터 승인을 받으셨다면 PR을 병합해주세요.

@river20s river20s added the java label Dec 14, 2024
@river20s river20s requested a review from o-ddong December 14, 2024 13:09
@river20s river20s self-assigned this Dec 14, 2024
@river20s river20s requested a review from a team as a code owner December 14, 2024 13:09
@HC-kang
Copy link
Contributor

HC-kang commented Dec 14, 2024

@river20s 님 안녕하세요!
rebase 후 통합 과정에서 올려주신 PR이 다시 실패하였는데, 현재 제출해주신 풀이의 마지막에 개행문자가 누락되어있는것으로 보입니다!
각 파일의 마지막 줄에 개행문자를 추가 해 주시면 정상적으로 통합하실 수 있습니다!

@river20s
Copy link
Contributor Author

@HC-kang 님 감사합니다! 답안 제출 가이드를 다시 한 번 읽어보며 문제를 해결하고 있습니다.

Copy link
Contributor

@HC-kang HC-kang left a comment

Choose a reason for hiding this comment

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

@river20s 님, 첫주차 고생하셨습니다!
한줄 한줄 명확하게 흐름을 적어주신 부분이 좋네요

가능하시다면 다음 주차부터는 시간, 공간복잡도 분석과 의견도 작성해주시면 다른 분들과 상호 리뷰가 더 풍성해지지 않을까 싶습니다!

위쪽에 안내 드린 대로 개행문자를 추가해 주시면 병합 가능하실거에요. 만약 또 문제가 생긴다면 말씀해주세요!

}
}

return false;
Copy link
Contributor

Choose a reason for hiding this comment

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

@river20s 보통 이런 부분은 리뷰로 남기진 않는데, indent 정리가 필요해보입니다! 🧹

@river20s river20s merged commit 2dbf19c into DaleStudy:main Dec 14, 2024
1 check passed
@river20s river20s changed the title [river20s] WEEK 01 solutions [river20s] WEEK 01 Dec 14, 2024
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.

2 participants