Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:86400
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: error (pb1.pair.com: domain belski.net from 85.214.73.107 cause and error)
To: "'Internals'" <internals@lists.php.net>
Date: Wed, 27 May 2015 15:29:28 +0200
Message-ID: <063f01d09881$2bbf12d0$833d3870$@belski.net>
MIME-Version: 1.0
Content-Type: text/plain;
	charset="us-ascii"
Content-Transfer-Encoding: 7bit
Thread-Index: AdCYe+dK7v1H+glZS6CSIRkaLYRjdA==
Content-Language: en-us
Subject: Pull request labels handling
From: anatol.php@belski.net ("Anatol Belski")

Hi,

the PRs on github are already being labeled. Multiple labels are possible.
However the github ACLs lack on granularity
https://help.github.com/articles/permission-levels-for-an-organization-repos
itory/ . Due to that, while being a useful feature, labeling is only
available for project admins or alike. Now with PHP7 such labeling becomes
probably even more sense as we'll need to handle BC breaks and other
situations which would need more testing and care. So the amount of work to
evaluate and review PRs will grow while the "work team" probably not.

To improve this, here's an idea originally coming from Ferenc. And it is
simple - the labeling feature could be integrated with our systems using the
github API. That would allow to achieve more granularity on who is allowed
to label. AFAIR there was already some people willing to do labeling, so
that were a great possibility to build such a team. Such pre evaluating and
labeling could possibly prevent some awkward situations, help to sort out
potential issues and BC breaches.

Thus asking for opinions. As it stands completely open - does this idea
sound eligible, how can it be integrated, who is yet up for the pre
evaluation/labeling job? Or any other propositions.

Regards

Anatol