New Manager In A Strange Land. I don't remember it being that short.
Something we've gone back and forth a couple times on the team is how strict we should be with checkins AKA commits. Much like a society swings back and forth between law and anarchy, the team swings back and forth between a strict checkin policy and a lax one. With the lax policy, everybody complains about the build being broken. With a strict policy, everybody complains about the time they have to spend validating their data, and how they can't check in because the build has been locked.
Our current policy is on the strict side. FWIW:
- when the build is broken we lock the depot - only people who are attempting to fix the build can commit
- the depot is locked an hour before the end of core hours, so people cannot "commit and leave"
- people who break the build and don't manage to fix it immediately are put on the blacklist; for the next week, they can only commit if they have permission
- a few special people are on the "awesome" list - they can commit at any time for any reason.
Despite the strict policy the build is frequently broken on at least one of the consoles for long periods of time, but it hasn't been in effect that long. We've also upgraded our build machines so they can churn faster - it could take an hour for the machine to detect a break, and another hour before the machine would validate the fix.