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

Git-Flow vs GitHub-Flow #28

Merged
merged 7 commits into from
Jul 29, 2024
Merged

Git-Flow vs GitHub-Flow #28

merged 7 commits into from
Jul 29, 2024

Conversation

soi-ha
Copy link
Member

@soi-ha soi-ha commented Jul 22, 2024

저번주에 Git 브랜칭 전략에 대해 수업을 듣게되어 좀 더 자세히 알아보고자 아티클을 찾아 번역했습니다~!
사실 브랜칭 전략 듣기 전에 이미 저희 팀은 GitHub-Flow로 정하긴 했었는데요.. 읽다보니 저희 팀의 브랜칭 전략은 GitHub-Flow와 Git-Flow를 짬뽕...?해서 한 느낌이네용..
뭐 각자 팀마다 맞는 걸로 변경시키면 되는 거니까 ~~~

@soi-ha soi-ha requested a review from Jaymyong66 July 22, 2024 15:05
@soi-ha soi-ha requested a review from llqqssttyy July 23, 2024 01:16
@soi-ha soi-ha self-assigned this Jul 23, 2024
@soi-ha soi-ha requested review from Jaymyong66 and removed request for Jaymyong66 and llqqssttyy July 23, 2024 01:17
Copy link
Member

@Jaymyong66 Jaymyong66 left a comment

Choose a reason for hiding this comment

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

간단한 리뷰를 남겼습니다~
저희 조도 git-flow에서 약간 변형한 형태로 사용 중인데, 각자의 상황에 맞게 쓰는게 가장 좋은 방법이라고 생각해요. (main - develop - dev/fe, dev/be - 각자의 브랜치)

고생하셨습니당!


## Git-Flow

Git-Flow는 서로 다른 기능 간의 병렬 개발 요구를 해결하는 방법을 제공합니다. 기능을 개발하기 시작하면 master 브랜치에서 feature 브랜치를 생성합니다. 그런 다음 기능에 대한 모든 개발 작업은 이 feature 브랜치에서 수행됩니다. 기능 작업을 완료한 후에는 릴리스를 위한 주 코드 경로로 feature 브랜치를 다시 병합합니다.
Copy link
Member

Choose a reason for hiding this comment

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

글의 첫문장이 잘 안 읽혀서 다음과 같이 살짝 의역하면 어떨까요? 꼭 반영 안해도 될 것 같습니당

Suggested change
Git-Flow는 서로 다른 기능 간의 병렬 개발 요구를 해결하는 방법을 제공합니다. 기능을 개발하기 시작하면 master 브랜치에서 feature 브랜치를 생성합니다. 그런 다음 기능에 대한 모든 개발 작업은 이 feature 브랜치에서 수행됩니다. 기능 작업을 완료한 후에는 릴리스를 위한 코드 경로로 feature 브랜치를 다시 병합합니다.
Git-Flow는 서로 다른 기능의 병렬적인 개발에 대한 요구를 해소하는 방법을 제공합니다. 기능을 개발하기 시작하면 master 브랜치에서 feature 브랜치를 생성합니다. 그런 다음 기능에 대한 모든 개발 작업은 이 feature 브랜치에서 수행됩니다. 기능 작업을 완료한 후에는 릴리스를 위한 main 코드 경로로 feature 브랜치를 다시 병합합니다.

- 개발자들은 개발 요청을 받아 develop 브랜치에서 feature 브랜치를 생성합니다.
- 개발자들은 로컬 개발 및 로컬 검증을 완료한 후 코드를 feature 브랜치에 제출합니다.
- 코드는 feature 브랜치에서 검증되며, 새로 생성된 코드를 지속적으로 병합합니다.
- 개발자들은 기능 개발을 완료하고 feature 브랜치에서 검증 중에 오류가 감지되지 않습니다. 그런 다음 기능 브랜치의 코드를 개발 브랜치에 병합합니다.
Copy link
Member

Choose a reason for hiding this comment

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

브랜치는 영어 단어로 통일하면 좋겠네요!

Suggested change
- 개발자들은 기능 개발을 완료하고 feature 브랜치에서 검증 중에 오류가 감지되지 않습니다. 그런 다음 기능 브랜치의 코드를 개발 브랜치에 병합합니다.
- 개발자들은 기능 개발을 완료하고 feature 브랜치에서 검증 중에 오류가 감지되지 않습니다. 그런 다음 feature 브랜치의 코드를 develop 브랜치에 병합합니다.

Copy link
Member Author

Choose a reason for hiding this comment

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

오... 한글로 한 부분이 있었네요?! 수정할게용


## GitHub-Flow

Git-Flow와는 달리 GitHub-Flow는 릴리스 브랜치를 사용하지 않습니다. GitHub-Flow의 개념에 따르면 버전이 준비되면 바로 배포할 수 있습니다. 마찬가지로 GitHub-Flow는 hotfix가 사소한 기능 변경과 동일하다고 믿으며, 처리 방법도 유사해야 한다고 생각합니다.
Copy link
Member

Choose a reason for hiding this comment

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

위와 같은 이유로!

Suggested change
Git-Flow와는 달리 GitHub-Flow는 릴리스 브랜치를 사용하지 않습니다. GitHub-Flow의 개념에 따르면 버전이 준비되면 바로 배포할 수 있습니다. 마찬가지로 GitHub-Flow는 hotfix가 사소한 기능 변경과 동일하다고 믿으며, 처리 방법도 유사해야 한다고 생각합니다.
Git-Flow와는 달리 GitHub-Flow는 release 브랜치를 사용하지 않습니다. GitHub-Flow의 개념에 따르면 버전이 준비되면 바로 배포할 수 있습니다. 마찬가지로 GitHub-Flow는 hotfix가 사소한 기능 변경과 동일하다고 믿으며, 처리 방법도 유사해야 한다고 생각합니다.

- **master 브랜치의 모든 코드는 배포할 수 있는 최신 작업 버전입니다.**
- 새로운 작업을 수행하려면 master 브랜치에서 새 브랜치를 만들고 목적을 명확하게 나타내기 위해 명시적으로 이름을 지정합니다. 예를 들어 새로운 일정 전략이라는 이름을 지정합니다.
- 코드 변경 사항은 가능한 한 자주 로컬 브랜치에 커밋해야 합니다. 한편 변경 사항은 가능한 한 자주 서버의 동일한 브랜치 이름의 브랜치와 동기화해야 합니다.
- 새 코드를 master 브랜치에 병합하려면 **코드 검토를 요청하기 위해 풀 요청을 시작**해야 합니다.
Copy link
Member

Choose a reason for hiding this comment

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

사소) 풀이 full 이라면? ㅋㅋㅋㅋㅋㅋㅋ

Suggested change
- 새 코드를 master 브랜치에 병합하려면 **코드 검토를 요청하기 위해 요청을 시작**해야 합니다.
- 새 코드를 master 브랜치에 병합하려면 **코드 검토를 요청하기 위해 pull 요청을 시작**해야 합니다.

@Jaymyong66 Jaymyong66 merged commit cfef464 into main Jul 29, 2024
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