Re: [PATCH] dim: Enforce review requirements

2017-05-30 Thread Neil Armstrong
On 05/24/2017 11:28 AM, Daniel Vetter wrote: > We can't check this when applying (since r-b/a-b tags often get added > afterwards), but we can check this when pushing. This only looks at > patches authored by the pusher. > > Also update the docs to highlight that review requirements hold >

Re: [PATCH] dim: Enforce review requirements

2017-05-26 Thread Sean Paul
On Wed, May 24, 2017 at 11:28:12AM +0200, Daniel Vetter wrote: > We can't check this when applying (since r-b/a-b tags often get added > afterwards), but we can check this when pushing. This only looks at > patches authored by the pusher. > > Also update the docs to highlight that review

Re: [PATCH] dim: Enforce review requirements

2017-05-26 Thread Daniel Vetter
On Thu, May 25, 2017 at 7:37 AM, Lukas Wunner wrote: > On Wed, May 24, 2017 at 11:28:12AM +0200, Daniel Vetter wrote: >> We can't check this when applying (since r-b/a-b tags often get added >> afterwards), but we can check this when pushing. This only looks at >> patches

Re: [PATCH] dim: Enforce review requirements

2017-05-24 Thread Lukas Wunner
On Wed, May 24, 2017 at 11:28:12AM +0200, Daniel Vetter wrote: > We can't check this when applying (since r-b/a-b tags often get added > afterwards), but we can check this when pushing. This only looks at > patches authored by the pusher. BTW, can we also have a rule that large drivers (i.e. with

RE: [PATCH] dim: Enforce review requirements

2017-05-24 Thread Deucher, Alexander
aul; Daniel Vetter > Subject: [PATCH] dim: Enforce review requirements > > We can't check this when applying (since r-b/a-b tags often get added > afterwards), but we can check this when pushing. This only looks at > patches authored by the pusher. > > Also update the docs to

Re: [PATCH] dim: Enforce review requirements

2017-05-24 Thread Lukas Wunner
On Wed, May 24, 2017 at 11:28:12AM +0200, Daniel Vetter wrote: > We can't check this when applying (since r-b/a-b tags often get added > afterwards), but we can check this when pushing. This only looks at > patches authored by the pusher. > > Also update the docs to highlight that review

[PATCH] dim: Enforce review requirements

2017-05-24 Thread Daniel Vetter
We can't check this when applying (since r-b/a-b tags often get added afterwards), but we can check this when pushing. This only looks at patches authored by the pusher. Also update the docs to highlight that review requirements hold especially also for bugfixes. Cc: Patrik Jakobsson