• HubertManne@kbin.social
    link
    fedilink
    arrow-up
    12
    ·
    1 year ago

    I hate places without code review. Makes me nervous as heaven to merge code and want as many double checks as possible.

    • Noumena@kbin.social
      link
      fedilink
      arrow-up
      3
      ·
      edit-2
      1 year ago

      I feel this way about open source and the seemingly frequent lack of detailed code reviews. This one project had two function options to use from a library. One handles errors by returning them to the caller so they can be handled gracefully. The other, calls PANIC! They chose the latter and it causes a crash loop for a relatively easy to hit code condition that is sensitive to User input.

      Why ask for unit test, in the code review, when you can just accept the contribution for a feature that is used in large corps.

      • HubertManne@kbin.social
        link
        fedilink
        arrow-up
        2
        ·
        1 year ago

        I will say I hate having to pester folks to do review on a merge. The place im currently at requires two and its pulling teeth to get folks to look at someone elses code. well except for me I always treat merges as priority as that is ready to go whereas with mine if I have not merged then its not that ready.

  • rodhlann@kbin.social
    link
    fedilink
    arrow-up
    8
    ·
    1 year ago

    I hate code reviews because for whatever stupid reason my brain decides this is how it is.

    I was at an XP shop for a few years before my current job and I forgot how much I hate code reviews because we were constantly pairing and didn’t do reviews in the same way. Coming back to the standard sort of code review model has been torture.

  • AnarchoGravyBoat@kbin.social
    link
    fedilink
    arrow-up
    2
    ·
    1 year ago

    @Lenguador

    Code reviews are only bad until you realize you can just say “cool story bro” and move on.

    No change is always an acceptable response to review comments imo, unless it’s one of mine.

    If you can’t defend that turd of a code unit, why the hell do you think it deserves to go into production?

  • TechyShishy@kbin.social
    link
    fedilink
    arrow-up
    2
    ·
    1 year ago

    “I don’t understand what these two lines of code do, could you please modify, explain, or document.” is very different from “I don’t like the semantics of this function, replace it with this other one that does something unrelated.”

    We’ve worked at both places, and one is useful, the other… not so much.