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

[정기적 문제 풀이 추가 PR] 2022.03.30 정기적 문제 풀이 추가합니다! #8

Merged
merged 12 commits into from
Mar 30, 2022

Conversation

codeisneverodd
Copy link
Owner

총 7 문제 풀이하였습니다!

@codeisneverodd codeisneverodd self-assigned this Mar 30, 2022
@codeisneverodd
Copy link
Owner Author

README에서 conflict이 있어 해결하였습니다!
@jaewon1676 다른 내용 충돌 없는지 확인후 확인 되셨으면 merge 부탁드립니다!
코드리뷰를 하실 사항이 있다면 코드에 주석을 달으셔도 좋습니다 👍

Copy link
Owner Author

@codeisneverodd codeisneverodd left a comment

Choose a reason for hiding this comment

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

리드미 수정이 필요해보입니다. 다른 곳에는 문제를 찾을 수 없네요. 고생하셨습니다 😄

@@ -22,14 +22,14 @@
### Level 2 👨🏻‍💻(풀이 중..)

- 전체 문제 수: 64문제
- 풀이 문제 수: 44문제(2022.03.24)
- 풀이 완료 예상 시점: 2022년 3월 중
- 풀이 문제 수: 43문제(2022.03.28)
Copy link
Owner Author

Choose a reason for hiding this comment

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

문제 수정이 필요할 것 같습니다 :)

README.md Outdated
- 풀이 문제 수: 44문제(2022.03.24)
- 풀이 완료 예상 시점: 2022년 3월 중
- 풀이 문제 수: 43문제(2022.03.28)
- 풀이 완료 예상 시점: 2022년 4월 중

### Level 3 👨🏻‍💻(풀이 중..)

- 전체 문제 수: 52문제
- 풀이 문제 수: 1문제(2022.03.24)
Copy link
Owner Author

Choose a reason for hiding this comment

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

풀이 문제 수 수정이 필요해보입니다.

Copy link
Contributor

Choose a reason for hiding this comment

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

5문제로 수정해야겠네요 확인 했습니다.

Copy link
Owner Author

Choose a reason for hiding this comment

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

반영하겠습니다!

Copy link
Contributor

@jaewon1676 jaewon1676 left a comment

Choose a reason for hiding this comment

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

문제마다 주석을 잘 남겨주셨네요 다른분들에게도 도움이 될 것같습니다!

README.md Outdated
- 풀이 문제 수: 44문제(2022.03.24)
- 풀이 완료 예상 시점: 2022년 3월 중
- 풀이 문제 수: 43문제(2022.03.28)
- 풀이 완료 예상 시점: 2022년 4월 중

### Level 3 👨🏻‍💻(풀이 중..)

- 전체 문제 수: 52문제
- 풀이 문제 수: 1문제(2022.03.24)
Copy link
Contributor

Choose a reason for hiding this comment

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

5문제로 수정해야겠네요 확인 했습니다.


size() {
return this.rear - this.front
}
Copy link
Contributor

Choose a reason for hiding this comment

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

고생하셨습니다. 저도 참고하겠습니다..! :)


if (priorities[0] < Math.max(...priorities)){
if (priorities[0] < Math.max(...priorities)) {
Copy link
Contributor

Choose a reason for hiding this comment

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

줄 정리 감사합니다 ㅎㅎ

@codeisneverodd codeisneverodd merged commit 8291459 into main Mar 30, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants