Striving to become a master Go/Cloud developer; Father ๐จโ๐งโ๐ฆ; ๐ค/((Full Stack Web|Unity3D) + Developer)/g; Science supporter ๐ฉโ๐ฌ; https://coder.today
Because people are ..humans, why do we have laws and punishments, why do we have concrete walls in the middle of the road ....
You can enforce most of the rules with automatic tools.
You cannot do a PR until the commit goes trough all the linters and tests with success.
You cannot do a PR without tests.
You cannot merge a PR without a 1-2 reviewers approval, which should cover the rest.
The guy who you describe clearly does not belong in that team (if this behavior is not an exception), but I've heard about team leaders that did those things, so ... though luck.
And if the majority of the team, and most important the leadership already agreed on coding and quality standards, you do not need to enforce the rules nor is a matter of self improve, it is a matter or professionalism and bad behavior, and both of them are reasons to get fired.
For further actions, you may consider blocking this person and/or reporting abuse
We're a place where coders share, stay up-to-date and grow their careers.
Because people are ..humans, why do we have laws and punishments, why do we have concrete walls in the middle of the road ....
You can enforce most of the rules with automatic tools.
You cannot do a PR until the commit goes trough all the linters and tests with success.
You cannot do a PR without tests.
You cannot merge a PR without a 1-2 reviewers approval, which should cover the rest.
The guy who you describe clearly does not belong in that team (if this behavior is not an exception), but I've heard about team leaders that did those things, so ... though luck.
And if the majority of the team, and most important the leadership already agreed on coding and quality standards, you do not need to enforce the rules nor is a matter of self improve, it is a matter or professionalism and bad behavior, and both of them are reasons to get fired.