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

행사 기획, 팀원 모집 부분 내용 추가 #35

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

loreenkim
Copy link
Contributor

행사 기획, 팀원 모집 부분 내용 추가했습니다. :)

행사 기획, 팀원 모집 부분 내용 추가했습니다. :)
@loreenkim loreenkim requested review from harrydrippin and a team August 17, 2020 05:54
@loreenkim loreenkim added this to the 1. 준비 milestone Aug 17, 2020
@loreenkim loreenkim removed this from the 1. 준비 milestone Aug 17, 2020
@loreenkim loreenkim added the 문서 작성 문서에 새 내용을 추가했거나 수정했어요. label Aug 17, 2020
Copy link
Member

@harrydrippin harrydrippin left a comment

Choose a reason for hiding this comment

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

감사합니다. 내용에는 이견이 없습니다만, 아래 내용을 확인해주세요.

@@ -4,15 +4,30 @@

## I. 준비

### 앰배서더 모집
Copy link
Member

Choose a reason for hiding this comment

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

기존에 ## I. 준비 항목에 있던 모든 Heading을 삭제하시고 새로운 Heading 2개를 삽입하셨습니다. 혹시 의도하신 수정일까요?

제 생각에는 작성하신 내용들이 ### 행사 기획(새로 만들어짐)과 ### 앰배서더 모집(기존에 있었음) 하위에 나뉘어 들어가고, 기존의 Heading이 유지되어야 하는 것이 아닌가 싶습니다.

GUIDE.md Outdated Show resolved Hide resolved
GUIDE.md Outdated Show resolved Hide resolved
GUIDE.md Outdated Show resolved Hide resolved
loreenkim and others added 3 commits August 17, 2020 15:10
Co-authored-by: Seunghwan Hong <[email protected]>
Co-authored-by: Seunghwan Hong <[email protected]>
@harrydrippin
Copy link
Member

추가로, 현재 변경 사항이 #2#3 의 내용을 모두 포함하고 있다고 하기에는 내용이 많이 추가되어야 할 것 같습니다. 이 브랜치에서 계속 진행하실 생각이신가요?

만약 여러 브랜치에 나누어 작업하려고 하신다면, Linked Issues에서 이 Issue들을 제외해야 합니다. 지금 그대로 이 PR이 Merge된다면 #2#3 이 Close 되어서 예기치 못하게 작업이 완료된 것으로 착각할 수 있을 것 같아요.

@loreenkim
Copy link
Contributor Author

  1. 추가로, 현재 변경 사항이 1. 준비 / 앰배서더 모집 / 선발 방식 #21. 준비 / 섭외 / 소개 문서 제작, 섭외 타임라인 #3 의 내용을 모두 포함하고 있다고 하기에는 내용이 많이 추가되어야 할 것 같습니다. 이 브랜치에서 계속 진행하실 생각이신가요?
    => 네! 일단 1차 저장하는 느낌으로 넣어두었는데요. 혹시 여기서 계속 작업해도 될까요?

  2. 두번째 주신 질문에 대해서는 잘 모르겠습니다.

  • 브랜치를 어떻게 나눈것이 좋을지 방법
  • issue는 설정하는 방법

뭐가, 동시에 협업할때 좋은 방법인지 잘 모르겠습니다. 방법을 제안해주실 수 있을까요?

@harrydrippin
Copy link
Member

harrydrippin commented Aug 17, 2020

  1. 추가로, 현재 변경 사항이 1. 준비 / 앰배서더 모집 / 선발 방식 #21. 준비 / 섭외 / 소개 문서 제작, 섭외 타임라인 #3 의 내용을 모두 포함하고 있다고 하기에는 내용이 많이 추가되어야 할 것 같습니다. 이 브랜치에서 계속 진행하실 생각이신가요?
    => 네! 일단 1차 저장하는 느낌으로 넣어두었는데요. 혹시 여기서 계속 작업해도 될까요?

네, 여기서 쭉 작업하신 후 링크한 이슈들이 전부 만족될 때 Merge하면 될 것 같습니다.

  1. 두번째 주신 질문에 대해서는 잘 모르겠습니다.
  • 브랜치를 어떻게 나눈것이 좋을지 방법
  • issue는 설정하는 방법

뭐가, 동시에 협업할때 좋은 방법인지 잘 모르겠습니다. 방법을 제안해주실 수 있을까요?

Branch는 구분된 Task 하나당 하나 정도로 유지하는 것이 가장 좋습니다. 모든 Branch는 Issue를 하나 이상 해결하는 형태로 가야 하고, 항상 PR은 특정 Issue와 엮여있어야 합니다.

만약 이 Branch에서 저 Issue들이 해결될 때까지 쭉 작업하려고 하셨다면 현재 상태 그대로 가는 것이 좋을 것 같습니다. 다만 여러 개로 쪼개질 것 같다고 판단된다면 Linked Issue를 조절하셔서 이 PR에서 해결할 Issue만 남겨두고 뜯어내시는 것이 가장 좋은 방법일 것 같아요. @soojungsueme

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
문서 작성 문서에 새 내용을 추가했거나 수정했어요.
Projects
None yet
2 participants