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

[jinho] week 07 #957

Merged
merged 1 commit into from
Jan 27, 2025
Merged

[jinho] week 07 #957

merged 1 commit into from
Jan 27, 2025

Conversation

neverlish
Copy link
Contributor

@neverlish neverlish commented Jan 26, 2025

답안 제출 문제

체크 리스트

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

@neverlish neverlish requested a review from bus710 January 26, 2025 06:03
@neverlish neverlish requested a review from a team as a code owner January 26, 2025 06:03
@neverlish neverlish changed the title [jinho[] week 07 [jinho] week 07 Jan 26, 2025
@github-actions github-actions bot added the go label Jan 26, 2025
@neverlish
Copy link
Contributor Author

요즘 바빠서 일요일이 되어서야 문제를 조금씩 풀고 있네요 😅 여유를 찾게 되면 일찍 풀어보려고 합니다
리뷰해주실 분 미리 감사 드립니다 (__)

Copy link
Contributor

@TonyKim9401 TonyKim9401 left a comment

Choose a reason for hiding this comment

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

jinho 님 안녕하세요 :)
이번주는 7주차가 아니라 8주차라 문제들 재확인 부탁드릴겠습니다!

@neverlish
Copy link
Contributor Author

jinho 님 안녕하세요 :) 이번주는 7주차가 아니라 8주차라 문제들 재확인 부탁드릴겠습니다!

확인 감사합니다!
그러면 이 PR을 닫고 다른 PR로 열어도 괜찮을까요?

@bus710
Copy link
Contributor

bus710 commented Jan 27, 2025

테스트 코드까지 다 작성하는 정성...!
감사합니다.

@neverlish
Copy link
Contributor Author

일단 푼 것은 병합하고, 8주차 PR 을 별도로 올리겠습니다! 감사합니다

@neverlish neverlish merged commit 2496829 into DaleStudy:main Jan 27, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Completed
Development

Successfully merging this pull request may close these issues.

3 participants