Skip to content

Conversation

HerrineKim
Copy link
Contributor

답안 제출 문제

체크 리스트

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

@HerrineKim HerrineKim requested a review from a team as a code owner December 13, 2024 12:06
@github-actions github-actions bot added the js label Dec 13, 2024
@HerrineKim HerrineKim requested a review from HodaeSsi December 13, 2024 12:06
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.

모든 문제를 잘 푸셨네요. 시간복잡도만 분석해주셨는데 공간복잡도까지 분석해주시면 금성첨화일 것 같습니다. linelint 체크가 실패하고 있어서 제가 승인해드려도 PR 병합이 안 되실 거에요.아래의 링크를 확인해 주세요! #664 게시물 참고하셔서 조치 부탁드리겠습니다.

@HerrineKim
Copy link
Contributor Author

모든 문제를 잘 푸셨네요. 시간복잡도만 분석해주셨는데 공간복잡도까지 분석해주시면 금성첨화일 것 같습니다. linelint 체크가 실패하고 있어서 제가 승인해드려도 PR 병합이 안 되실 거에요.아래의 링크를 확인해 주세요! #664 게시물 참고하셔서 조치 부탁드리겠습니다.

리뷰 감사합니다! 다음 주차부터는 꼭 포함시켜 볼게요 :)

@rivkode rivkode mentioned this pull request Dec 14, 2024
9 tasks
@HerrineKim HerrineKim merged commit 575eb87 into DaleStudy:main Dec 14, 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.

2 participants