Phabricator + 'Reviewed by' [was Re: svn commit: r278472 - in head/sys: netinet netinet6]

[Bryan Drewery]

On 2/9/2015 3:45 PM, Bjoern A. Zeeb wrote:
>>  Commented upon by hiren and sbruno
>>  See Phabricator D1777 for more details.
>>
>>  Commented upon by hiren and sbruno
>>  Reviewed by:	adrian, jhb and bz
> I have not reviewed this;  as a matter of fact you are aware that I still wanted to do that.
> 

Something about Phabricator is not jiving with our commit terminology.
This has happened before as well with other commits. I'm sure everyone
is good-intentioned as well.

There's not 1 person on D1777 who has 'accepted' it. That is what
warrants a 'Reviewed by' to me.

It's clear to me, but seems unclear to others. I really think the
reviewer list needs to be split up. Rather than using icons, use
separate lists. Reviewers requested: accepted: commented: changes
requested:.



-- 
Regards,
Bryan Drewery

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 473 bytes
Desc: OpenPGP digital signature
URL: <http://lists.freebsd.org/pipermail/svn-src-all/attachments/20150213/032dd755/attachment.sig>