Date: Mon, 20 Nov 2017 20:32:31 -0500 From: Ed Maste <emaste@freebsd.org> To: Alan Somers <asomers@freebsd.org> Cc: src-committers <src-committers@freebsd.org>, svn-src-all@freebsd.org, svn-src-head@freebsd.org Subject: Re: svn commit: r326039 - in head: etc/mtree usr.bin/head usr.bin/head/tests Message-ID: <CAPyFy2BDB5dVF_hURt9Xs4U4PzHW8KoL=6%2BEf0WGJgTxQSVaPw@mail.gmail.com> In-Reply-To: <201711202255.vAKMt2nP014201@repo.freebsd.org> References: <201711202255.vAKMt2nP014201@repo.freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
On 20 November 2017 at 17:55, Alan Somers <asomers@freebsd.org> wrote: > Author: asomers > Date: Mon Nov 20 22:55:02 2017 > New Revision: 326039 > URL: https://svnweb.freebsd.org/changeset/base/326039 > > Log: > Add ATF tests for head(1) > > Submitted by: Fred Schlecter <https://github.com/fjs-github>; > Reviewed by: asomers, jilles > MFC after: 3 weeks > Differential Revision: https://github.com/freebsd/freebsd/pull/127 It's certainly a good idea to capture the source of changes that come in via GitHub pull requests, but the 'Differential Revision' field is specifically for reviews in our own hosted Phabricator instance. We don't have an officially established metadata field for changes that come in via pull requests at the moment. I'd suggest we just go with: Pull request: https://github.com/freebsd/freebsd/pull/127
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAPyFy2BDB5dVF_hURt9Xs4U4PzHW8KoL=6%2BEf0WGJgTxQSVaPw>