Versions Compared

Key

  • This line was added.
  • This line was removed.
  • Formatting was changed.

...

  • We'll not use hotfix branches
  • We'll not use development/integration branches (for now -> see table under OpenIssues)

Master

  • Rule: revert immediately as soon as it fails
  • no direct commits except minor enhancement (fixing typos & formatting etc.) 
    • make sure to inform in M5 Review chat

Feature branches

  • on central repo
  • remove them after the final merge
  • list of feature branches 
    • --> things we are actively working on or are waiting
  • frequently merge from master to feature branches to reduce last-minute conflicts

Maintenance

  • no changes
  • no hot fix branches (for the moment)

Merging (feature-branch -> master)

  • only after review by the Black Knightsreview 
  • we use  git merge --no-ff instead
    • no fastforward - avoids losing information about the historical existence of a feature branch + groups together all commits of a feature

(other option would be git merge --squash when merging from feature branch to master -> one consolidated commit with meaningful message for a feature)

Not using one of the above will result in situations where it might be very hard to figure out what commit's belong together (e.g. really bad if u have to revert a feature)

Open Issues

Decision Table
aspectlocal feature branchesremote feature branches
collaborate(minus) more effort - map other users repo(plus) as usual
remove branch(plus) just delete it locally(minus) all (or at least some) devs need permission
see what's worked on(minus) not visible on git.magnolia-cms...(plus) easily visible
backup work(minus) has to be considered(plus) as usual

 

...