I was in a performance review recently and my team leader, who is a non-technical person was going through the usual: what do you feel has gone well so far this year, what do you think could have gone better, etc.
I was then completely caught off guard in that my team leader said they had gone back through our version control history (presumably with the aid of a technical person in the team) and was looking through all of our previous pull requests for this performance review period to see what comments had been left by our peers on each of our pull requests. As a result, my team lead said that across a number of my PR's that similar comments on fixes/improvements had been identified and as a result, they were marking my performance for this period as needing improvement. They admitted they hadn't read the code itself as it wouldn't have meant anything to them, but by reading the comments which had been left by my peers, they could make that judgment call.
For information, my team lead is very new to the management role but has been in the team longer than I have (which has been a couple of years).
This is the first time in my career that my performance as a developer has been called into question based on this criteria, I was wondering if this was commonplace outside of my current place of employment, or is this a bit of a red flag that something else may be going on?