We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
feature/34 -> develop으로 merge를 시도하는 과정에서 auto merge가 불가함을 인지
feature/34
develop
feature/25
stockMapper.xml
StockMapper.java
cherry-pick
git reset
resolve/34
충돌 해결 완료~
관련 commit message에도 해당 이슈를 기록.
resolve 정책은 팀마다 다를 수 있습니다.
The text was updated successfully, but these errors were encountered:
feat: Add getHeadStockByItemId for fb32f21 (#34)(#39)
getHeadStockByItemId
fb32f21
fc5702d
이번 이슈를 통해 충돌 관리 및 브랜치 전략의 중요성을 느꼈습니다. 향후 작업 시 충돌 가능성을 줄이기 위해 작은 단위로 더 자주 머지, 브랜치 간 의존성 최소화 등의 개선 방안을 논의하면 좋겠습니다.
Sorry, something went wrong.
맞아요! 실제로 이러한 오픈소스 프로젝트들은 리뷰가 매우 신중하고 오래 걸리기 때문에 한 브랜치의 작업 단위를 크게 잡지 않는 것이 conflict 가능성을 줄이는 방법 중 하나입니다.
상당히 생산적이고 저에게도 생각의 여지를 주는 좋은 코멘트네요. 감사합니다!
Queue-ri
No branches or pull requests
문제상황
feature/34
->develop
으로 merge를 시도하는 과정에서 auto merge가 불가함을 인지브랜치 상태
2d7137f 와 c8137e7 이 서로 충돌하는 것으로 추정됨
원인
feature/25
)의 리뷰가 비교적 늦어짐feature/34
는feature/25
가 merge되지 않은develop
의 revision에서 파생feature/34
에서feature/25
의 변경사항에 포함된 파일을 수정feature/34
를 그대로 merge할 수 없게 됨해결방안
feature/34
에서 conflict가 발생하는 파일은 stock 도메인의 mapper 파일임을 확인 (stockMapper.xml
,StockMapper.java
)cherry-pick
git reset
feature/34
로 재지정하고 force pushdevelop
을 pull한 후resolve/34
브랜치 생성resolve/34
를 본 이슈와 연결하여 PR결과
충돌 해결 완료~
참고
관련 commit message에도 해당 이슈를 기록.
resolve 정책은 팀마다 다를 수 있습니다.
The text was updated successfully, but these errors were encountered: