Skip to content

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

Open
loreenkim wants to merge 4 commits into
masterfrom
soojung/first-step-for-hackathon-1
Open

행사 기획, 팀원 모집 부분 내용 추가#35
loreenkim wants to merge 4 commits into
masterfrom
soojung/first-step-for-hackathon-1

Conversation

@loreenkim

Copy link
Copy Markdown
Contributor

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

행사 기획, 팀원 모집 부분 내용 추가했습니다. :)
@loreenkim
loreenkim requested review from a team and harrydrippin 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

@harrydrippin harrydrippin left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

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

Comment thread GUIDE.md

## I. 준비

### 앰배서더 모집

Copy link
Copy Markdown
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이 유지되어야 하는 것이 아닌가 싶습니다.

Comment thread GUIDE.md
Comment thread GUIDE.md Outdated
Comment thread GUIDE.md Outdated
@harrydrippin
harrydrippin requested a review from a team August 17, 2020 06:01
loreenkim and others added 3 commits August 17, 2020 15:10
Co-authored-by: Seunghwan Hong <harrydrippin@gmail.com>
Co-authored-by: Seunghwan Hong <harrydrippin@gmail.com>
Co-authored-by: Seunghwan Hong <harrydrippin@gmail.com>
@harrydrippin

Copy link
Copy Markdown
Member

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

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

@loreenkim

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

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

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

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

@harrydrippin

harrydrippin commented Aug 17, 2020

Copy link
Copy Markdown
Member
  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

Development

Successfully merging this pull request may close these issues.

1. 준비 / 앰배서더 모집 / 선발 방식 1. 준비 / 앰배서더 모집 / 모집 글, 모집 방법, 모집 홍보

2 participants