I'd agree that micromanaging is a bad sign. Too often I get micromanaging about style critiques like "turn this into one if statement instead of nested ones and it's more clear", but absolutely no comments on the meat of a significant refactor. To be clear, I think style rules are important, but this is the same guy who writes 150+ character lines with ternaries like they're going out of style.
