현재 위치

홈 ›› Forums ›› 공부 ›› 프로그래밍 QnA

[완료] github에서 코드 contribute 할때 말인데요.

nomail의 이미지
글쓴이: nomail / 작성시간: 일, 2014/03/16 - 8:13오후

git과 github을 배우고 있는 초보입니다.

하나 궁금한게 있어서 잘 아시는 분들께 여쭤볼려는데요.

github에서 어떤 프로젝트를 fork하여 수정한 코드를 contribute를 할려고 합니다.

최종 작업 후 원저작자에게 pull request 할때,

제쪽 저장소의 master 브랜치를 보내줘야 하는지(master에는 원저작자의 master/develop 브랜치에 제가 작업한 결과물을 rebase 함)

아니면 다른 topic브랜치를 만들어 거기서 작업 후 topic 브랜치를 알려줘야 하는지요?

보통은 어떤 방식으로 진행되는지 궁금합니다.

Forums: 
프로그래밍 QnA
klara의 이미지

글쓴이: klara / 작성시간: 월, 2014/03/17 - 8:58오전

포크가 순전히 그 풀리퀘스트를 위한 거라면 마스터로 해도 문제는 없는데, 이 마스터가 계속 작업해야하는 거라면 따로 브랜치를 만드는게 좋습니다. 한번 풀리퀘스트를 하면 그 풀리퀘스트가 닫힐때까지 해당 브랜치에 커밋되는 내용들까지 자동으로 갱신되어 풀리퀘스트에 포함되기 때문입니다.

nomail의 이미지

글쓴이: nomail / 작성시간: 월, 2014/03/17 - 9:40오전

답변 주셔서 고맙습니다.
관심있는 프로젝트가 있어서 git이랑 github을 배우고 있는데요.
pull request 받는 쪽에서는 아무래도 merge 히스토리가 있는 것보단 최종 rebase 된 브랜치를 받는것이 좋겠단 생각이 들었습니다.
여유가 되면 계속해서 contribute를 할 예정인데 그럼 프로젝트 fork 하고나서 제 쪽의 master브랜치는 원본 저장소와 merge하는 용도로 두고
제가 작업하는 develop브랜치, 그리고 작 작업들 히스토리를 남길 topic브랜치들.. topic브랜치는 코드 수정이 완료 될때마다 develop브랜치에 merge 하고
그리고 최종적으로 pull request 보낼 때는 topic브랜치로 rebase한 master브랜치를 보내줄려고 합니다.
제가 답변을 잘 이해한게 맞는지요?

klara의 이미지

글쓴이: klara / 작성시간: 월, 2014/03/17 - 10:07오전

보통은 마스터가 주작업 공간이 되고(말그대로 마스터), 풀리퀘스트를 날릴 필요가 있을 때마다 정리해서 브랜치를 만들어서 날립니다.
그리고 작업 내용은 자동으로 기록되어 남으므로 단순히 작업 히스토리를 위해서 브랜치를 나눌 필요는 없습니다.

무엇보다 그냥 한번 해보시면 바로 감이 올겁니다. 풀리퀘스트 잘못 날린다고 성질내는 개발자는 (별로) 없습니다.
실수로 잘못 날렸으면 그냥 닫고 다시 날리면 됩니다.
그리고 최종적으로 등록하기 전에 어떤 내용의 풀리퀘스트가 날아가는지 커밋 로그와 파일 변경 사항까지 깃헙에서 전부 확인 가능하니까 전부 완벽하게 계획하고 하시려하지 말고 일단 이것저것 눌러보면서 해보시는게 좋습니다.

nomail의 이미지

글쓴이: nomail / 작성시간: 월, 2014/03/17 - 10:26오전

네 말씀 고맙습니다.
pro git 책 내용중에 오픈소스 프로젝트에서는 원 저작자의 커밋 룰을 지켜야 욕을 안 먹는다는 얘기가 있었는데.. 우선 말씀대로 한번 도전해봐야 겠습니다.
친절한게 많이 알려주셔서 정말 고맙습니다^^

댓글 달기

텍스트 포맷에 대한 자세한 정보

Filtered HTML

  • 텍스트에 BBCode 태그를 사용할 수 있습니다. URL은 자동으로 링크 됩니다.
  • 사용할 수 있는 HTML 태그: <p><div><span><br><a><em><strong><del><ins><b><i><u><s><pre><code><cite><blockquote><ul><ol><li><dl><dt><dd><table><tr><td><th><thead><tbody><h1><h2><h3><h4><h5><h6><img><embed><object><param><hr>
  • 다음 태그를 이용하여 소스 코드 구문 강조를 할 수 있습니다: <code>, <blockcode>, <apache>, <applescript>, <autoconf>, <awk>, <bash>, <c>, <cpp>, <css>, <diff>, <drupal5>, <drupal6>, <gdb>, <html>, <html5>, <java>, <javascript>, <ldif>, <lua>, <make>, <mysql>, <perl>, <perl6>, <php>, <pgsql>, <proftpd>, <python>, <reg>, <spec>, <ruby>. 지원하는 태그 형식: <foo>, [foo].
  • web 주소와/이메일 주소를 클릭할 수 있는 링크로 자동으로 바꿉니다.

BBCode

  • 텍스트에 BBCode 태그를 사용할 수 있습니다. URL은 자동으로 링크 됩니다.
  • 다음 태그를 이용하여 소스 코드 구문 강조를 할 수 있습니다: <code>, <blockcode>, <apache>, <applescript>, <autoconf>, <awk>, <bash>, <c>, <cpp>, <css>, <diff>, <drupal5>, <drupal6>, <gdb>, <html>, <html5>, <java>, <javascript>, <ldif>, <lua>, <make>, <mysql>, <perl>, <perl6>, <php>, <pgsql>, <proftpd>, <python>, <reg>, <spec>, <ruby>. 지원하는 태그 형식: <foo>, [foo].
  • 사용할 수 있는 HTML 태그: <p><div><span><br><a><em><strong><del><ins><b><i><u><s><pre><code><cite><blockquote><ul><ol><li><dl><dt><dd><table><tr><td><th><thead><tbody><h1><h2><h3><h4><h5><h6><img><embed><object><param>
  • web 주소와/이메일 주소를 클릭할 수 있는 링크로 자동으로 바꿉니다.

Textile

  • 다음 태그를 이용하여 소스 코드 구문 강조를 할 수 있습니다: <code>, <blockcode>, <apache>, <applescript>, <autoconf>, <awk>, <bash>, <c>, <cpp>, <css>, <diff>, <drupal5>, <drupal6>, <gdb>, <html>, <html5>, <java>, <javascript>, <ldif>, <lua>, <make>, <mysql>, <perl>, <perl6>, <php>, <pgsql>, <proftpd>, <python>, <reg>, <spec>, <ruby>. 지원하는 태그 형식: <foo>, [foo].
  • You can use Textile markup to format text.
  • 사용할 수 있는 HTML 태그: <p><div><span><br><a><em><strong><del><ins><b><i><u><s><pre><code><cite><blockquote><ul><ol><li><dl><dt><dd><table><tr><td><th><thead><tbody><h1><h2><h3><h4><h5><h6><img><embed><object><param><hr>

Markdown

  • 다음 태그를 이용하여 소스 코드 구문 강조를 할 수 있습니다: <code>, <blockcode>, <apache>, <applescript>, <autoconf>, <awk>, <bash>, <c>, <cpp>, <css>, <diff>, <drupal5>, <drupal6>, <gdb>, <html>, <html5>, <java>, <javascript>, <ldif>, <lua>, <make>, <mysql>, <perl>, <perl6>, <php>, <pgsql>, <proftpd>, <python>, <reg>, <spec>, <ruby>. 지원하는 태그 형식: <foo>, [foo].
  • Quick Tips:
    • Two or more spaces at a line's end = Line break
    • Double returns = Paragraph
    • *Single asterisks* or _single underscores_ = Emphasis
    • **Double** or __double__ = Strong
    • This is [a link](http://the.link.example.com "The optional title text")
    For complete details on the Markdown syntax, see the Markdown documentation and Markdown Extra documentation for tables, footnotes, and more.
  • web 주소와/이메일 주소를 클릭할 수 있는 링크로 자동으로 바꿉니다.
  • 사용할 수 있는 HTML 태그: <p><div><span><br><a><em><strong><del><ins><b><i><u><s><pre><code><cite><blockquote><ul><ol><li><dl><dt><dd><table><tr><td><th><thead><tbody><h1><h2><h3><h4><h5><h6><img><embed><object><param><hr>

Plain text

  • HTML 태그를 사용할 수 없습니다.
  • web 주소와/이메일 주소를 클릭할 수 있는 링크로 자동으로 바꿉니다.
  • 줄과 단락은 자동으로 분리됩니다.
댓글 첨부 파일
이 댓글에 이미지나 파일을 업로드 합니다.
파일 크기는 8 MB보다 작아야 합니다.
허용할 파일 형식: txt pdf doc xls gif jpg jpeg mp3 png rar zip.
CAPTCHA
이것은 자동으로 스팸을 올리는 것을 막기 위해서 제공됩니다.