Branching structure #658
Unanswered
WeilerP
asked this question in
Method/Algorithm
Replies: 1 comment
-
|
This has been implemented in a slightly different form in #725. |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
-
@michalk8 (and FYI @Marius1311), I'm continuing the discussion regarding branching structure from #655 - thought it might be best to have this in a separate discussion rather than buried in a PR.
I think
cellrank@devandcellrank@master"diverged" becausecellrank@devwas squash merged intocellrank@master. Let's saycellrank@devis three commits ahead ofcellrank@master. If you thencellrank@devintocellrank@master, GitHub will say thatcellrank@devis three commits ahead and one commit behindcellrank@mastercellrank@devintocellrank@master, GitHub will say thatcellrank@devis one commit behindcellrank@master(that's the merge commit)Right now, GitHub says that
cellrank@masterandcellrank@devare even. This is the case because you merged (merge commit)cellrank@masterintocellrank@dev, right?When merging
cellrank@devintocellrank@master, we should, IMO, be using a simple merge commit, for features, bug fixes, etc. squash merges. This way,cellrank@mastermimicscellrank@dev, i.e. the individual feature commits are still visible in its commit history and you can still rungit blameand get reasonable results.Concerning branch names: I personally use
feat/X(features, enhancements, etc.) is based off the develop branch and squash merged into it again.fix/X(bug fixes) is based off the develop branch and squash merged into it again.hotfix/X(hotfixes after which new release is tagged) is based offmasterand squash merged both inmasteranddevelop.release/X.X.X(releases) is based offdevelopand merged (merge commit) both intomasteranddevelop(in case something was added to the release after branching offdevelop. You canThe CI can be configured to add a new tag based on the branch name, i.e. for
release/X.X.Xthe new release/tagX.X.Xis added. The same goes for hotfixes.Beta Was this translation helpful? Give feedback.
All reactions