You have to check if it works. But if it works… what is anyone bitching about?
They’re bitching about him pushing untested (by his own words) and broken code straight to main instead of going through the proper “PR, Review, Merge” loop like anyone else.
If you “have to check if it works” it should be in a PR for people to play with it, suggest improvements and make changes, not directly in the codebase.
They’re bitching about him pushing untested (by his own words) and broken code straight to main instead of going through the proper “PR, Review, Merge” loop like anyone else.
If you “have to check if it works” it should be in a PR for people to play with it, suggest improvements and make changes, not directly in the codebase.
They’d already admitted they accidentally pushed to the wrong branch and cleaned it up.