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

[Round1] Step3 - sy #998

Closed
wants to merge 1 commit into from

Conversation

jeseoyoung
Copy link

step3.swift:

  • function 내 parameter에서 불필요한 단어를 제외
  • 중괄호 컨벤션 추가
  • printPeperoBodyAndToppings()에서 공백을 출력하도록 개선
  • setPeperoBody()의 함수명을 loopForBodyPrint()로 개선(해당 함수명이 적절한지 궁금합니다!)

플로우차트:

  • 격자 제거 및 단순화

나만의 빼빼로 출력:

  • png 파일을 추가하였습니다.

@jaemuYeo

@jaemuYeo
Copy link

앗 혹시 이전에 보낸 PR에 이어서 PR재요청해볼 수 있을까요?
지금 전달해주신 방식은 재요청이 아닌 새롭게 PR을 보내준 것이여서요!!

@jeseoyoung jeseoyoung force-pushed the ss_20_sy_0919 branch 4 times, most recently from f43ec6a to c322801 Compare September 20, 2024 07:30
@jeseoyoung
Copy link
Author

앗 혹시 이전에 보낸 PR에 이어서 PR재요청해볼 수 있을까요? 지금 전달해주신 방식은 재요청이 아닌 새롭게 PR을 보내준 것이여서요!!

제가 여태껏 수정하는 법을 제대로 모르고 있었네요😂 git commit --amend를 통해서 하는 게 맞을까요?!

@jaemuYeo
Copy link

에고 코멘트를 늦게 확인했네요 ..
해당 방법도 괜찮습니다! 같은 브랜치로 PR이 여러개 날라올경우 Merge 충돌이 발생할 수 있어요 ㅠㅠ
PR 재용청 또는 PR 수정 등으로 구글링하시면 여러 방법 나올거예요 :)
깃헙 응답이 늦으면 디스코드로 연락 남겨주시면 더 빠르게 확인하겠습니다!

@jeseoyoung
Copy link
Author

git commit --amend을 통해 기존 PR을 수정했습니다!([Round1] Step3 - sy #997)
확인 부탁드려요

@jeseoyoung jeseoyoung closed this Sep 23, 2024
@jaemuYeo
Copy link

네엡 확인했습니다!! 반영하시느라 고생많으셨습니다 👍
내일 오전 중 리뷰 남겨두겠습니다!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants