3 ms·
> Blocking is a weapon of last resort This is exactly how I feel. The only times I would block a PR is if I know it will either: a- actually break something e
by rkomorn 1y ago
> Blocking is a weapon of last resort
This is exactly how I feel. The only times I would block a PR is if I know it will either:
a- actually break something existing
b- not deliver what the PR says it should deliver (eg new feature but the feature doesn't actually work)
It's basically saying "I'm certain that if we merge this, something bad will happen and we'll either cause an incident or need a hotfix".
- hinkley 1y agoc - it represents a complete failure to adhere to a standard of coding that we all approved as a group. Any of these things are a negative commentary on your process, or the members of your team. When one finds oneself standing in a hole, it is important to stop digging.
- rkomorn 1y agoAgreed on c- . The rest seems a bit strident. Stuff happens. I wouldn't exactly go running to the incident/retro unless there was repeated pattern.