-
Notifications
You must be signed in to change notification settings - Fork 0
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
Nifty git PR+merge commit #7
Comments
@dandv I deleted my fork. |
Hm. I've deleted my fork too but that didn't result in that sort of nice history. |
The history you see in github commits does not show the branching. My current merge strategy is to wait until I want to release, then I start pulling the code from all PRs, rebase on top of develop (might edit manually some commits to make them fit) and then merge. I have a script that does all that, allowing me to edit if stuff goes wrong. The small photo of me is because I edited the commits, but I'm not the original author. Also you shouldn't merge the PR commit with the merge -- there should be a merge commit. Do this to check out the real history: git log --graph --decorate --pretty=oneline --abbrev-commit |
Hey @davidsonfellipe, how did you do this ?
The text was updated successfully, but these errors were encountered: