minus-squareAlfa@lemmy.worldtoProgramming@programming.dev•Other dev won't follow best practices, is this common?linkfedilinkarrow-up18·1 year agoIt’s not called ‘PR Approval’ it’s called PR Review for a reason. Developer should fix broken test especially if there’s super fragile stuff involved ( and that should be fixed asap). linkfedilink
It’s not called ‘PR Approval’ it’s called PR Review for a reason. Developer should fix broken test especially if there’s super fragile stuff involved ( and that should be fixed asap).