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.

Sign up for GitHub

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

Jump to bottom

[브런치 및 PR Merge 규칙] #7

Open

linehyang opened this issue Oct 28, 2021 · 0 comments

Contributor

linehyang commented Oct 28, 2021 •

edited

Loading

Melthing-Brain Main branch에는 merge 금지
merge시에는 dev brach(default)에 merge
원본repo를 remote repo로 Fork한 후
remote repo에서 원본 repo로 push 및 PR
branch명
-feat/기능이름 : 기능 구현시 해당 branch명으로 branch를 따서 작성
ex) 로그인 구현시 해당 branch로 작성 feat/signin
-fix-bug/기능이름 : 버그 수정시 해당 branch명으로 branch를 따서 수정
ex) 로그인 페이지 버그 수정시 fix-bug/signin

주의사항

브런치 새로 만들었을 경우 원본 repo와 싱크 맞추고 remote repo에 push 하기
git pull [remote] [dev]
git push [remote명] [branch]
만약 npm install한게 있으면 merge하고 PR할때 반드시 npm install 다시 해야한다고 명시해주기

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment