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

[구문영] Week1 문제 풀이 #311

Merged
merged 8 commits into from
Aug 19, 2024
Merged

[구문영] Week1 문제 풀이 #311

merged 8 commits into from
Aug 19, 2024

Conversation

@dev-jonghoonpark
Copy link
Contributor

image

마지막 라인에 개행 넣어주시면 좋을 것 같습니다.
일반적으로 ide나 에디터에 옵션이 있습니다.

이유는 아래 링크 보시면 좋을 것 같습니다 : )
https://stackoverflow.com/questions/729692/why-should-text-files-end-with-a-newline

@yolophg
Copy link
Contributor

yolophg commented Aug 12, 2024

안녕하세요, 코치 Helena 입니다.
아래 사진과 같이 Projects 에서 project와 iteration 설정도 같이 부탁드립니다!
이번엔 제가 해두었는데, 원할한 스터디 그룹 진도 관리와 통계를 위해서 중요한 부분이니 다음 이터레이션부터는 해주시면 감사드리겠습니다 :)
Screenshot 2024-08-12 at 11 48 15
Screenshot 2024-08-12 at 11 41 08

@taekwon-dev taekwon-dev added the js label Aug 17, 2024
@GUMUNYEONG GUMUNYEONG marked this pull request as ready for review August 17, 2024 17:56
@GUMUNYEONG GUMUNYEONG requested a review from seona926 August 17, 2024 18:01
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.

수고 많으셨습니다. 3문제 푸신 것도 잘 하신 거라고 생각합니다. 모임 전까지 PR 병합 부탁드리겠습니다.

};

// TC: O(n)
// SC: O(n)
Copy link
Contributor

Choose a reason for hiding this comment

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

마지막에 line break 없는 파일이 메인 브랜치에 유입되지 않도록 각별히 주의바랍니다. (참고: #334)

};

// TC: O(log n)
// SC: O(log n)
Copy link
Contributor

@DaleSeo DaleSeo Aug 17, 2024

Choose a reason for hiding this comment

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

공간 복잡도 분석이 정확한지 재고해보시면 좋을 것 같아요!

@GUMUNYEONG GUMUNYEONG merged commit 8c4fb50 into DaleStudy:main Aug 19, 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.

5 participants