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

[codyman0] Week 01 Solutions #330

Merged
merged 7 commits into from
Aug 22, 2024
Merged

[codyman0] Week 01 Solutions #330

merged 7 commits into from
Aug 22, 2024

Conversation

CodyMan0
Copy link
Contributor

@CodyMan0 CodyMan0 commented Aug 17, 2024

@CodyMan0 CodyMan0 added the py label Aug 17, 2024
@CodyMan0 CodyMan0 self-assigned this Aug 17, 2024
@CodyMan0 CodyMan0 marked this pull request as draft August 17, 2024 05:17
return False
if sortedArray[i] == sortedArray[i + 1] :
return True
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.

추가 문제풀이에 앞서 라인브레이크 설정 부탁드립니다! 보시면 현재 파일 맨 아래 빨간색 원 표시가 보이실거예요! 해당부분 수정 및 남은 4문제는 추가해서 커밋 부탁드리겠습니다 😄

Copy link
Contributor Author

Choose a reason for hiding this comment

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

넵!

@CodyMan0 CodyMan0 marked this pull request as ready for review August 20, 2024 13:32
@DaleSeo
Copy link
Contributor

DaleSeo commented Aug 20, 2024

  • 금주 1주차 문제는 2주차 주말에 따라가겠습니다.

스터디 1기 때 보면 전주에 못 푸신 문제까지 다음 주에 모두 다 푸시려고 무리하시다가 오히려 포기하게 되시는 분들이 많았습니다. 그냥 2주차에는 새로운 문제에 집중하는 것도 나쁘지 않을 것 같습니다 :)

Copy link
Contributor

@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.

v2.1.0 릴리즈 시점에 이 PR은 병합 준비가 안 되서 포함시키지 못했습니다. ㅠㅠ 그룹 스터디의 일정 관리를 위해서 이번 주 부터는 기한에 맞춰서 꼭 PR 병합 좀 부탁드리겠습니다.

@DaleSeo
Copy link
Contributor

DaleSeo commented Aug 21, 2024

@CodyMan0 님, 혹시 PR을 오픈 상태로 두시는 이유가 있으신가요? linelint 이슈도 없고 병합하셔도 될 것 같은데요?

@CodyMan0
Copy link
Contributor Author

CodyMan0 commented Aug 22, 2024 via email

@CodyMan0 CodyMan0 merged commit 8f47a65 into DaleStudy:main Aug 22, 2024
2 checks 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.

3 participants