you are viewing a single comment's thread
view the rest of the comments
[–] 71 points 11 months ago (6 children)

Git commits with message saying “pushing changes” and there are over 50 files with unrelated code in it.

  • source
  • parent
  • hideshow 6 child comments
  • [–] 4 points 11 months ago (1 child)

    In the past I had commit messages with change numbers from a system, that was no longer in use.

    So the commit just said “CH-12345“. It is the kind of annoying, where you can't even really be mad at someone.

  • source
  • parent
  • hideshow 1 child comment
  • [–] 2 points 11 months ago

    I put my ticket numbers in my tickets, but i also try to describe the change too (e.g. “Fix bug where xyz happens due to zyx action”). Also, atomic commits: commit only related changes.

    Yes, it takes longer to commit large changes, BUT you can easily merge the commits, and rollback only what needs to be rolled back.

  • source
  • parent