I'm the half part of a 2 dev team. Last year it took me around 3 months of internal discussions between us devs and the project manager to get us to use a PR t

I'm the half part of a 2 dev team. Last year it took me around 3 months of internal discussions between us devs and the project manager to get us to use a PR template in Github (The PM was more or less sold on the idea). Commit messages (Not mine, obviously) rarely amount to more than "Fixed <this>". And don't get me started on code comments (Non-existent, obviously). The other coder is responsible for around 90% of the new code we ship these days (The product is "frozen" because there is a better product developed by a different and larger team, but we still have clients on this one) and I also had to strong arm him into adding type hints for vars and functions (In PHP) because he is very fond of patterns (Think, SomeObjectFactoryFactoryFactory) so it would make development with PhpStorm easier. So now the code is polluted with /** * var $sometint integer */ that explain absolutely nothing.