Common sense is never common. I'm sure you can find something written to block nonsense like that - is it fully unit tested? You do have something written that requires unit tests right?
But at the end of the day, if you're on a team with somebody like that then you NEED written rules to deal with that nonsense and the Code Review process is the first step to identify it.
The nonsense gets merged the first time, but not before you add a written rule against it going forward. Problem solved.
But at the end of the day, if you're on a team with somebody like that then you NEED written rules to deal with that nonsense and the Code Review process is the first step to identify it.
The nonsense gets merged the first time, but not before you add a written rule against it going forward. Problem solved.