We had a non-tech lead at some point (although his title was something like development manager) and he would praise my coworker for how many check-ins he did. Except that coworker would do things like:
1- Copy-paste an entire class into a new class and change a single constant in it, because he was too lazy to do inheritance.
2- "Solve" multiple bugs a day that he had introduced himself the day before.
3- Loudly complain about other people's frameworks/codes.
He was the super confident type even though he was wrong more often than not. But paired with a non-tech lead with his own impostor syndrome, it was a recipe for disaster.
1- Copy-paste an entire class into a new class and change a single constant in it, because he was too lazy to do inheritance.
2- "Solve" multiple bugs a day that he had introduced himself the day before.
3- Loudly complain about other people's frameworks/codes.
He was the super confident type even though he was wrong more often than not. But paired with a non-tech lead with his own impostor syndrome, it was a recipe for disaster.