Template for pull requests

Ryan Schmidt ryandesign at macports.org
Sun Jan 15 22:57:14 UTC 2017


> On Jan 15, 2017, at 02:44, Mojca Miklavec <mojca at macports.org> wrote:
> 
> Hi,
> 
> I've recently seen some "templates" in our pull requests, for example on
>    https://github.com/macports/macports-ports/pull/158
> 
> Were those added manually?

We don't have a PULL_REQUEST_TEMPLATE.md in our repo, so yes.

> What do others think of adding some templates to our repos?

I often find them annoying in other projects because they ask the wrong questions. Can we find the right questions to ask?

> Mojca
> 
> On 22 December 2016 at 20:56, Mojca Miklavec wrote:
>> Hi,
>> 
>> I have an impression that we are now getting pull requests from
>> upstream developers (or some close-to-upstream users/developers) who
>> never contributed to MacPorts earlier and sometimes don't even have
>> MacPorts installed and never test the changes. Some of them submit
>> both to MacPorts and HomeBrew at the same time.
>> 
>> Well, this is both good and bad at the same time. It's good because we
>> are getting code from upstream in a timely matter. It's slightly bad
>> because these patches are not tested (even though the software itself
>> might be tested on Mac). Of course the one who commits a pull request
>> is also expected to test the changes, but I would find it helpful if
>> we had a pull request template where submitters could mark whether
>> they:
>> 
>> - tested the changes and if so, on which OS versions
>> - ran `port lint`
>> - whether this (a) comes from maintainer, (b) is non-maintainer or
>> doesn't require permission, or (c) requires permission from
>> maintainer; in that case @mention it
>> 
>> ... probably more
>> 
>> I'm not saying that we should force every singe upstream contributor
>> to do all the possible tests, but it would be at least helpful to know
>> in case they didn't.
>> 
>> Mojca
>> 
>> (HB has a template with a checklist for example:
>> https://github.com/Homebrew/brew/pulls)



More information about the macports-dev mailing list