search

pep-dortmund / homepage-toolbox

Website for the PeP et al. Toolbox Workshop at TU Dortmund
http://toolbox.pep-dortmund.org
3 stars 1 forks source link

Revert pull.rebase option to false #50

Closed VukanJ closed 5 years ago

maxnoe commented 5 years ago

Warum?

VukanJ commented 5 years ago
  1. Es geht um das Verhalten von git bei Merge Konflikten. Normalerweise löst man einen Konflikt mit git add und git commit. Mit pull.rebase = true, muss man einen rebase machen und kommt dann mit großer Wahrscheinlichkeit durcheinander wenn man online tutorials befolgt und einen extra commit nach git add erstellt. Wenn man sich nicht auskennt, ist dann nämlich nicht so klar wo man dann gelandet ist und probiert dann Stackoverflow Befehle aus bis was kaputt geht
  2. Rebasing verschleiert an welchem Punkt ein Konflikt gelöst wurde und vorallem wie und durch wen
  3. Rebasing ändert die commit chronologie (nicht so kritisch, sollte aber erwähnt werden)

Der vorteil ist natürlich die etwas linearere historie. Ich finde es jedenfalls nicht unwichtig diesen Punkt irgendwie anzusprechen, oder zumindest in den git Folien zu kennzeichnen, dass git rebase --continue beim lösen von Konflikten vom Standard abweicht. In diesem Fall kann man diesen request schließen

maxnoe commented 5 years ago

Ja, ist vermutlich besser den default auf merge zu lassen und nur zu sagen, dass man es verändern kann

bixel commented 5 years ago

Finde ich gut! Ich meine wir hatten das auch schon mal nach der letzten git Toolbox session bequatscht. So ist der Einstieg in git noch etwas harmloser.

LuckyJosh commented 5 years ago

@bixel "Harmloser Einstieg in git" klingt für mich nach einem Oxymoron :sweat_smile: