• deadbeef79000@lemmy.nz
    6·
    11 months ago

    P1000x this.

    I can read a diff.

    I need to know why.

    No, a code comment isn’t good enough, it’s out of date after the next commit.

    • DontTakeMySky@lemmy.world
      1·
      11 months ago

      Code comments for "why"s that persist. Commits for why’s that are temporary.

      If you need to run X before Y, add a comment. If you added X before why because it was easier, leave it in a commit

        • DontTakeMySky@lemmy.world
          2·
          11 months ago

          With a comment on the test detailing why it matters so people don’t just assume the test is out of date when it fails.

          And ideally test the underlying result of x before y, not the fact that x is called before y.

          And while we’re at it, assert in Y that X has been called, and again comment the reason for the preconditions.