Skip to main content
2 of 5
added 285 characters in body
Stephan Branczyk
  • 59.3k
  • 29
  • 129
  • 210

You've already reversed the changes. So let it go.

That being said, the next time this happens. Be sure that there is no wiggle room with your interpretation of the coding conventions. Because, if you're going to have a battle with someone, be sure to pick a battle that you have a 100% chance of winning.

This is important especially if you do not have as much social capital with management as this person has already.

And so, take your time, get familiar with the code base. Once you find something worthwhile that you can change about his code that he can't complain about. And once you have nothing else more important to do. Then, do it.

And if he does complain to you, then you can just tell him to fight on changing the code conventions, and that once he receives approval for those new coding conventions he has in mind, that you will be glad to take a look at them and implement them, but not before then.

And if the argument is about something that is not written in the code conventions, bring a copy of Code Complete 2nd edition to work and use that as your final arbiter.

Also, you should probably ask management to organize weekly code reviews. After all, if you have a chance to discuss coding decisions during code reviews. It will give you a safer and less confrontational way to understand and/or challenge some of your peers bad habits.

Stephan Branczyk
  • 59.3k
  • 29
  • 129
  • 210