Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:78546
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain zend.com designates 209.85.217.182 as permitted sender)
MIME-Version: 1.0
In-Reply-To: <CAEZPtU6PA1=jNPcQB5n4m1gYvfxBh_5MsEPHmvD_gAS8hHDyWA@mail.gmail.com>
References: <EB032A39-EB24-4BBE-A2D3-14BF63809873@zort.net> <CAEZPtU6PA1=jNPcQB5n4m1gYvfxBh_5MsEPHmvD_gAS8hHDyWA@mail.gmail.com>
Date: Sat, 1 Nov 2014 22:27:46 +0800
Message-ID: <CABBJUpdyMW_9r-FkxtOC3a2e4TS2J+dNTXN3sOYhw+KxLYjB4g@mail.gmail.com>
To: Pierre Joye <pierre.php@gmail.com>
Cc: John Bafford <jbafford@zort.net>, PHP internals <internals@lists.php.net>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable
Subject: Re: [PHP-DEV] [RFC] GitHub Pull Requests Triage Team
From: laruence@php.net (Xinchen Hui)

Hey:



On Sat, Nov 1, 2014 at 7:53 PM, Pierre Joye <pierre.php@gmail.com> wrote:
> Hi,
>
> On Oct 31, 2014 4:57 AM, "John Bafford" <jbafford@zort.net> wrote:
>>
>> Hi,
>>
>> I would like to propose the creation of a team to triage the pull
> requests on GitHub, to help ensure that the pull requests are handled in =
a
> timely manner. I am also volunteering to lead such a team, should the RFC
> be approved.
>>
>> https://wiki.php.net/rfc/github-pr
>>
>> PHP=E2=80=99s GitHub repository has over 180 open pull requests. Many of=
 these
> are bug fixes or new tests that should be incorporated into PHP, but have
> not been because the PRs aren=E2=80=99t being regularly monitored. As a r=
esult, the
> large number of open pull requests may also be discouraging contributions=
,
> as potential contributors may see that pull requests are not being acted =
on
> and decline to submit changes.
>
> As much as I like the idea I never understood why we do not have them her=
e.
>
> Given that many PRs have discussions, it should show up on internals.
>
> PS: yes we have a PR list. Which did not work as expected. PRs and
> discussions in them should not be considered as noises to the internals l=
ist
> Cheers,
Maybe we should create some tags , like "minor", "critical", "need verify" =
etc..

for minors, anyone have time can merge it. (like typo fix).

thanks
> Pierre



--=20
Xinchen Hui
@Laruence
http://www.laruence.com/