Date: Wed, 1 Mar 2017 12:00:48 +0000 From: =?UTF-8?Q?Bart=C5=82omiej_Rutkowski?= <robak@freebsd.org> To: Kristof Provost <kp@freebsd.org> Cc: =?UTF-8?Q?Bart=C5=82omiej_Rutkowski?= <robak@freebsd.org>, Warner Losh <imp@bsdimp.com>, git@freebsd.org Subject: Re: pull requests Message-ID: <CAGFrfxa3LPY6VM0wbfCbvaOinKHGOCVCBEjbrLBMoFumNWg3zA@mail.gmail.com> In-Reply-To: <C6BF03E7-64B8-4183-9095-76C5757FAF3C@FreeBSD.org> References: <CANCZdfoT%2B9nHqS2qYuf_-sBAYDAzH=2ePW=ADPwGhjUG00S2Jw@mail.gmail.com> <CAGFrfxZRUWryg8nHQ2gGbrcZ1U1EgsaSVfpn_mgortmih99fAQ@mail.gmail.com> <C6BF03E7-64B8-4183-9095-76C5757FAF3C@FreeBSD.org>
next in thread | previous in thread | raw e-mail | index | archive | help
On Wed, Mar 1, 2017 at 11:52 AM, Kristof Provost <kp@freebsd.org> wrote: > On 1 Mar 2017, at 10:36, Bart=C5=82omiej Rutkowski wrote: > >> What kind of policy, regarding what and 'policing' what and how exactly? >> Who and how will enforce such policy and on whom? In the end, "we can't >> force anyone to do anything" so unless we commit to do something on >> GitHub, >> I'd see it as a dead law. >> >> There is an official FreeBSD account on GitHub and I don=E2=80=99t expec= t that to > go > away. > > See > https://github.com/freebsd/freebsd/pulls > > (As an aside, I note that someone has been cleaning these up. Awesome.) > > We are getting pull requests there and something should be done with them= . > > Not to mention I've got a feeling this is completely disconnected from th= e >> general direction that I think has been 'agreed', that is we'll have a >> software solution getting GH PR's into Phabric reviews and GH Issues int= o >> Bugzilla PR's automatically. What then, shouldn't it be more like Phabri= c >> reviews/Bugzilla PR policy instead? >> >> This is probably a good answer to the question of what to do with those > pull > requests. > > As the current situation is =E2=80=9Cthey=E2=80=99re completely ignored= =E2=80=9D anything would be > an > improvement. Even automatically closing them with =E2=80=9CGo to <foo> in= stead.=E2=80=9D > would > be an improvement. > > It might be nice to document a review policy (usable for GitHub, > Phabricator, > e-mail reviews, =E2=80=A6) somewhere. Things like =E2=80=9CHow long do I = wait for > reviews?=E2=80=9D, > =E2=80=9CWho do I subscribe/cc/=E2=80=A6?=E2=80=9D, =E2=80=9CWhat should = be done with abandoned reviews?=E2=80=9D =E2=80=A6 > > I fully support Warner=E2=80=99s efforts. > > Regards, > Krsitof > Don't get me wrong, I am simply trying to understand what exactly would that accomplish - see https://reviews.freebsd.org/D9649. Lets, for a fun thought experiment, imagine it was an GH pull request that was transferred to Phabric. What now? 2 weeks passed and nothing happened. Would any policy fix that? Kind regards, Bartek Rutkowski
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAGFrfxa3LPY6VM0wbfCbvaOinKHGOCVCBEjbrLBMoFumNWg3zA>