I am a mid level software developer in a small software company. I have been here for the last 8 months and due to my approach with code reviews I was added to the mandatory reviewers list early on.
I came here from a fairly large organization where the turnover rate was very low and every developer was extremely involved in the software development process. Coding standards were followed religiously and code quality was everything.
I came to this company to learn something different, being smaller company I thought I would learn more than I did back at my last job. Since, a few months, I have noticed a careless attitude among my team members when it comes to documentation, code quality and the approach towards code reviews is appalling. I being one of the reviewers on the code reviews, it is very obvious to me. I have to constantly remind them to follow the agreed upon code standards, to document stuff and just take ownership. I am not in a position to ask them to do it explicitly. Their attitude is starting to annoy me and almost stress me out. This attitude is same throughout all levels be it a junior developer or a more senior developer than me.
I cannot simply ignore the basic mistakes that are made in the reviews and let that code go in the production. One bad coder, but senior developer, came to me to ask me to back off on code reviews.
I have already talked to my manager (who is also their manager) about all this, but from his response it sounded to me like he has no control over them (he started a few weeks after me) so he is relatively new.
If you were in my situation, what would you do? Would you change your attitude to not care anymore?
Some e.g. of issues that I see in the code reviews:
Newly created unused methods/variables that are left as it is.
Naming conventions for new classes, properties, methods not followed. Properties for instance are left all lower case.
Obvious bugs are ignored and they get into arguments like the the back off comment.