Skip to content

Conversation

EgonD3V
Copy link
Contributor

@EgonD3V EgonD3V commented Aug 25, 2024

답안 제출 문제

Python

Swift

체크 리스트

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

@EgonD3V EgonD3V requested a review from HaJunYoo August 25, 2024 15:08
@EgonD3V EgonD3V self-assigned this Aug 25, 2024
@EgonD3V EgonD3V force-pushed the feat/week3 branch 2 times, most recently from ba96431 to 8ed3ebe Compare August 27, 2024 16:01
@EgonD3V EgonD3V marked this pull request as ready for review August 27, 2024 16:20
@EgonD3V EgonD3V requested a review from a team as a code owner August 27, 2024 16:20
@DaleSeo
Copy link
Member

DaleSeo commented Aug 28, 2024

코드 충돌 조치 부탁들립니다!

Shot 2024-08-27 at 22 13 18@2x

@DaleSeo DaleSeo requested a review from wogha95 August 28, 2024 02:16
@EgonD3V
Copy link
Contributor Author

EgonD3V commented Aug 28, 2024

코드 충돌 조치 부탁들립니다!

Shot 2024-08-27 at 22 13 18@2x

conflict 수정했습니다 감사합니다

@DaleSeo
Copy link
Member

DaleSeo commented Aug 28, 2024

5문제 푸셨다고 체크가 되어 있는데, 실제 파일은 3개 밖에 없는 것 같습니다. 그리고 신규 파일을 추가하신 것이 아니라 기존 파일을 변경하신 것으로 나오는데 혹시 지난 주에 미리 푸셔서 그런걸까요?

Shot 2024-08-28 at 18 43 08@2x

Shot 2024-08-28 at 18 42 55@2x

Shot 2024-08-28 at 18 44 44@2x

@EgonD3V
Copy link
Contributor Author

EgonD3V commented Aug 29, 2024

5문제 푸셨다고 체크가 되어 있는데, 실제 파일은 3개 밖에 없는 것 같습니다. 그리고 신규 파일을 추가하신 것이 아니라 기존 파일을 변경하신 것으로 나오는데 혹시 지난 주에 미리 푸셔서 그런걸까요?

@DaleSeo 네 그 지난 토요일에 5문제 다 작성은 하고, 나중에 몇 문제 수정했는데 force reset으로 깔끔하게 맞춰놓으려했는데 이미 merge되었더라고요; 다음부터는 빨리 풀면 stash해서 주차에 맞춰서 올리겠습니다.

그래도 일단 리뷰어께서 리뷰하시기 쉽도록 0a4e7f1 기준으로 5문제 풀이 맞춰놓았습니다

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.

수고하셨습니다!

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

Successfully merging this pull request may close these issues.

2 participants