|
|
|
|
|
by smurph
4945 days ago
|
|
I've had a lot of trouble with this at my current job. There's a strict policy that requires a review before anything makes it in to a deliverable branch, but there's also a culture of emergency fire fighting where they will expect problems fixed quickly, often outside of working hours when a code review isn't really possible. So far our team's done full blown reviews for all new components, but we've waived reviews for small bug fixes. One big hurdle is that we are using proprietary source control software that does not play nice with any of the established review tools (another battle I am trying to fight.). |
|