• Ephera@lemmy.ml
    link
    fedilink
    English
    arrow-up
    10
    ·
    1 day ago

    Worst part is when the un-refactored code continues to confuse other devs, meaning it keeps causing additional work, but you still don’t get time to actually fix it.

    • CarrotsHaveEars@lemmy.ml
      link
      fedilink
      arrow-up
      4
      ·
      1 day ago

      Commit1: actual change
      Commit2: this code is so misleading and I have to refactor it refactors
      Me: painfully rejects PR the story said nothing about refactoring…

      • collapse_already@lemmy.ml
        link
        fedilink
        English
        arrow-up
        1
        ·
        13 hours ago

        The mistake is admitting that you’re refactoring. I just implemented the feature/ fixed the bug. Why did I change so much? The issue was a little underscoped.

        The fun thing is to try to force your fellow devs to refactor piecemeal through comments on their pull requests.