It takes days to get a review. 3rd notice unanswered. So today it will be merged without review of 5 MRs. We’ll see what breaks in prod anyways.

  • 0
    You don't have merge rules for that?
  • 2
    I feel like reviews are just there to protect the company from malicious developers

    Pretty possible to write solid code without reviews. Maybe should have tests in the pipeline though
  • 0
    @ScriptCoded we do but they are freely editable.
  • 0
    If there's a rule to override reviews, M/PRs are pointless
Add Comment