• 0 Posts
  • 2 Comments
Joined 1 year ago
cake
Cake day: July 14th, 2023

help-circle
  • I find the “clean history” argument so flawed.

    Sure, if you’re they type to micro commit, you can squash your branch and clean it up before merging. We don’t need a dozen “fixed tests” commits for context.

    But in practice, I have seen multiple teams with the policy of squash merging every branch with 0 exceptions. Even going so far as squash merging development branches to master, which then lumps 20 different changes into a single commit. Sure, you can always be a git archeologist, check out specific revisions, see the original commits, and dig down the history over and over, to get the original context of the specific change you’re looking into. But that’s way fucking more overhead than just looking at an unmanipulated history and seeing the parallel work going on, and get a clue on context at a glance at the network graph.


  • Wrench@lemmy.worldtoMemes@lemmy.mlConfusing...
    link
    fedilink
    English
    arrow-up
    2
    ·
    1 year ago

    Except if you don’t know the full equation when you’re starting to write it. Most real world applications have you piecing things together as you go. Stopping and reordering it in an arbitrary “more readable” order is wasted work