Follow

"Please review this PR that contains enough changes to the API that you'll have to rewrite your own project."

:flan_angry:

*Makes a comment in the PR approval that this is a rubber stamp and it will take me weeks to know if this PR is awful or not*

Why approve a massive PR that I can't sensibly review? It's not my project, not my code. PR reviews are mandatory for master merges and the project can't move forward until someone approves the PR. because it changes the interface, my review is required.

Sign in to participate in the conversation
hackers.town

A bunch of technomancers in the fediverse. Keep it fairly clean please. This arcology is for all who wash up upon it's digital shore.