|
|
|
|
|
by bryanlarsen
5399 days ago
|
|
I'd be pretty leery of limiting who can send pull requests, but what Github needs to do is make it very clear that the big "merge pull request" button is usually a BAD idea unless it's a doc-only change. They should - create a branch or tag containing the merge so that it's easy to pull the change without having to add additional remotes. - provide instructions on how to pull, merge and push the request saying explicitly that the merge should be inspected and tested before doing the push. - make the latter more obvious to click than "merge pull request" button EDIT: I'm not saying that Linus needs this, but responding to Linus' comment about it being much too easy to do poor-quality commits. |
|