6

Nothing says I hate mondays more than reviewing a massive PR

Comments
  • 4
    Tell them to break it up into smaller digestible commits and respect your time.
  • 0
    @SortOfTested sadly it's a massive feature that got merged in after like 20 PRs and we are getting ready to release it to prod
  • 2
    @cyanide6033 The massive feature should consist of multiple, digestible commits.
  • 2
    I recommend moving away from these huge merges. Instead small changes often is the way to go. Embrace DevOps!
  • 0
    "Needs rework"
  • 1
    At least it's one related but massive feature. An ex coworker of mine loved to do large-scale refactorings while she was at it. In the years I was at that company she never understood why one could prefer to review these things separately.
Add Comment