Skip to content

10월 BE 블로그 포스팅 #235

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 6 commits into from
Nov 10, 2022
Merged

10월 BE 블로그 포스팅 #235

merged 6 commits into from
Nov 10, 2022

Conversation

@iju1633
Copy link

@iju1633 iju1633 commented Oct 19, 2022

No description provided.

@iju1633 iju1633 added the 🙋‍♂️🙋‍♀️ 리뷰 요청 PR 리뷰를 요청합니다~ label Oct 19, 2022
@iju1633 iju1633 requested a review from SSUHYUNKIM October 19, 2022 05:56
@iju1633
Update 2022-10-19-Logging.md
16308a6 
이미지 파일 형식 수정
@iju1633
Update archieve.png
6e0bac4 
이미지 파일 대체
@iju1633
Update 2022-10-19-Logging.md
eab8489 
[리뷰 불필요]
@SSUHYUNKIM SSUHYUNKIM requested a review from seunghb320 October 22, 2022 12:44
seunghb320
seunghb320 suggested changes Oct 27, 2022
View reviewed changes
Copy link

@seunghb320 seunghb320 left a comment •
edited
Loading

Choose a reason for hiding this comment

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

사소하거긴 한데 38번 라인의 오른쪽 'Print문'만 다른 라인과 다르게 P가 대문자인데 의도하신걸까요??
56번 라인 '쓰레드 별로 -> 쓰레드별로'가 맞는것 같습니다..!

@SSUHYUNKIM SSUHYUNKIM added ✅ 리뷰 완료 한 명 이상이 리뷰를 완료했을 때 🔨 수정 요청 리뷰 과정 중 수정 사항이 생겼을 때 and removed 🙋‍♂️🙋‍♀️ 리뷰 요청 PR 리뷰를 요청합니다~ labels Oct 27, 2022
@iju1633
Copy link
Author

iju1633 commented Oct 27, 2022 •
edited
Loading

사소하거긴 한데 38번 라인의 오른쪽 'Print문'만 다른 라인과 다르게 P가 대문자인데 의도하신걸까요??
56번 라인 '쓰레드 별로 -> 쓰레드별로'가 맞는것 같습니다..!

의도한 건 아니었네요ㅎㅎ P는 소문자로 고치고 문법도 수정 완료했습니다!

@SSUHYUNKIM SSUHYUNKIM added 🛠 수정 완료 수정 요청 사항이 모두 해결됐을 때 🙋‍♂️🙋‍♀️ 리뷰 요청 PR 리뷰를 요청합니다~ and removed ✅ 리뷰 완료 한 명 이상이 리뷰를 완료했을 때 🔨 수정 요청 리뷰 과정 중 수정 사항이 생겼을 때 labels Oct 27, 2022
seunghb320
seunghb320 approved these changes Oct 29, 2022
View reviewed changes
Copy link

@seunghb320 seunghb320 left a comment

Choose a reason for hiding this comment

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

고생하셨습니다! 테크톡을 통해 심화된 의견을 나눠본 주제를 모두에게 공유해주신 정말 좋은 글인것 같습니다!!

@SSUHYUNKIM SSUHYUNKIM added ✅ 리뷰 완료 한 명 이상이 리뷰를 완료했을 때 and removed 🛠 수정 완료 수정 요청 사항이 모두 해결됐을 때 🙋‍♂️🙋‍♀️ 리뷰 요청 PR 리뷰를 요청합니다~ labels Nov 1, 2022
@SSUHYUNKIM SSUHYUNKIM added the 🔨 수정 요청 리뷰 과정 중 수정 사항이 생겼을 때 label Nov 9, 2022
@SSUHYUNKIM
Copy link

SSUHYUNKIM commented Nov 9, 2022

재욱

  1. 혹시 예상한 부분과 다르게 된 부분이 있으면 확인 후 수정 부탁드립니다.
  2. 확인하셨을 수 있지만 맞춤법 검사기 한번 쭉 돌려서 수정해주시면 좋을 것 같습니다.
  3. (선택) 주요 단어에 강조 표시를 넣으면 좀 더 눈에 잘 보일 것 같습니다.

@iju1633
Copy link
Author

iju1633 commented Nov 10, 2022

확인 완료하였고 수정 사항 없습니다!

@SSUHYUNKIM SSUHYUNKIM added 💯 최종 완료 모든 수정과 리뷰가 끝난 후 merge를 기다릴 때 and removed ✅ 리뷰 완료 한 명 이상이 리뷰를 완료했을 때 🔨 수정 요청 리뷰 과정 중 수정 사항이 생겼을 때 labels Nov 10, 2022
@SSUHYUNKIM SSUHYUNKIM merged commit 22739c4 into GDSC-University-of-Seoul:master Nov 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@SSUHYUNKIM SSUHYUNKIM Awaiting requested review from SSUHYUNKIM

1 more reviewer

@seunghb320 seunghb320 seunghb320 approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@iju1633 iju1633

Labels

💯 최종 완료 모든 수정과 리뷰가 끝난 후 merge를 기다릴 때

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@iju1633 @SSUHYUNKIM @seunghb320