579
4 out of 5 developers (files.techhub.social)
submitted 1 year ago by to c/programmer_humor@programming.dev
5 comments fedilink hide all child comments
 

From: https://techhub.social/@FakeScrumStats/115246982974030476

all 5 comments

sorted by: hot top controversial new old
[–] 14 points 1 year ago

  • permalink
  • fedilink
  • source
    •
    [–] 7 points 1 year ago

    I am on a new team learning a new language and new frameworks, I can use all of the code review I can get. I need feed back, before I put a bunch of garbage into the code base. Code review is needed for new team members.

  • permalink
  • fedilink
  • source
    •
    [–] 4 points 1 year ago

    I'm the middle guy, but I'm a masochist, so:

    https://github.com/ZILtoid1991/pixelperfectengine

  • permalink
  • fedilink
  • source
    •
    [–] 2 points 1 year ago (1 child)

    The lead on my team is very much a LGTM reviewer. The project has been going on for 6-7 years.

  • permalink
  • fedilink
  • source
  • hideshow 1 child comment
    •
  • [–] 6 points 1 year ago* (last edited 1 year ago)

    This is how you get thousands of defects at worst and multiple engineers time worth of tech debt at best.

    We have a mantra that "If it's good enough to merge, it's good enough for production". With the intention of automatic production deployment sometime in the future.

    That's has helped us improve quality, a lot.

  • permalink
  • fedilink
  • source
  • parent