this post was submitted on 12 Feb 2024
165 points (94.1% liked)

Programmer Humor

32483 readers
395 users here now

Post funny things about programming here! (Or just rant about your favourite programming language.)

Rules:

founded 5 years ago
MODERATORS
 
you are viewing a single comment's thread
view the rest of the comments
[–] GissaMittJobb@lemmy.ml 32 points 9 months ago (7 children)

Taking the time to learn how git works on a conceptual level and then how to resolve merge conflicts effectively is a worthwhile investment. Highly recommended

[–] Kache@lemm.ee 2 points 9 months ago (6 children)

Even better, learn how to avoid conflicts from happening in the first place!

[–] pcouy@lemmy.pierre-couy.fr 3 points 9 months ago (5 children)

How do you avoid conflicts happening in the first place?

[–] Kache@lemm.ee 1 points 9 months ago* (last edited 9 months ago) (1 children)

It's kind of difficult to explain in the same way git is difficult to grok on the first try.

Perhaps it's convincing enough to just say:

  • Git is the fundamentally better at resolving merges/rebases without conflicts than older VCS that don't maintain a commit tree data structure.
  • Even within just git, using one diff algorithm vs another can mean the difference between git successfully merging vs failing and showing you a conflict
  • Software is flexible -- there are endless permutations to how it can be structured. Everything else being equal, some code/commit structures are more prone to conflicts than others

I.e. whether a conflict will happen is not some totally unpredictable random event. It's possible to engineer a project's code & repo so that conflicts are less common.

[–] pcouy@lemmy.pierre-couy.fr 1 points 9 months ago (1 children)

I can think of some "programming best practices" that can help with reducing merge conflicts, such as making small functions/methods, but I see it as a positive side effect.

I don't think avoiding merge conflicts should be a goal we actively try to reach. Writing readable code organized in atomic commits will already help you get fewer conflicts and will make them easier to resolve.

I've seen too many junior and students being distracted from getting their task done because they spent so much time "coordinating" on order to avoid these "scary" merge conflicts

[–] Kache@lemm.ee 2 points 9 months ago* (last edited 9 months ago)

avoiding merge conflicts

No, not like that -- you misunderstand. I'm not talking about actively avoiding conflicts. Coordinating to avoid merge conflicts is the same work as resolving a merge conflict anyway, just at a different time.

I'm talking about creating practices and environments where they're less likely to happen in the first place, never incurring the coordination cost at all.

One example at the individual level is similar to what you mentioned, but there's more to it. E.g. atomically renaming and moving in separate commits, so git's engine better understands how the code has changed over time and can better resolve merges without conflict.

But there're other levels to it, too. A higher-order example could be a hot module where conflicts frequently occur. Sure, atomic commits and all that can help "recover" from conflict more easily, but perhaps if the hot module were re-designed so that interface boundaries aligned with the domains of changes that keep conflicting, future changes would simply not conflict anymore.

IMO the latter has an actual productivity benefit for teams/orgs. Some portion of devs just aren't going to be that git proficient, and in this case, good high level organization is saving them from losing hours to incorrect conflict resolutions that can cause lost work, unintended logical conflicts (even though not lexical conflict), etc. Plus, it implies abstraction boundaries better match the changes demanded by the domain, so the code is likely easier to understand, too.

load more comments (3 replies)
load more comments (3 replies)
load more comments (3 replies)