[Openstack-docs] Voting of nicencess checks
Andreas Jaeger
aj at suse.com
Thu Jul 31 19:05:57 UTC 2014
On 07/31/2014 08:52 PM, Anne Gentle wrote:
>
>
>
> On Thu, Jul 31, 2014 at 3:26 AM, Christian Berendt
> <berendt at b1-systems.de <mailto:berendt at b1-systems.de>> wrote:
>
> Hello.
>
> At https://review.openstack.org/#/c/109559/ I proposed a review request
> to change the nicencess checks to vote for all our documents. At the
> moment all documents are clean. We normally -1 changes with a failed
> niceness check.
>
> Anne asked me to ask the list for their opinion.
>
>
> Yep, thanks Christian.
>
> If my memory serves, Tom was quite unhappy with the original whitespace
> gate as he struggled to help newcomers pass the gate. If we all think
> that problem is resolved, I'm fine with gating on whitespace
> infractions. I'm not a super fan, since it slows down patch merges, but
> if it's the way we're shifting as a team, I'm good with it.
We review this way already and I don't see major problems, so I'm fine
with enabling it. And if it really hurts us, we can reverse that also ;)
Andreas
--
Andreas Jaeger aj@{suse.com,opensuse.org} Twitter/Identica: jaegerandi
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Jeff Hawn,Jennifer Guild,Felix Imendörffer,HRB16746 (AG Nürnberg)
GPG fingerprint = 93A3 365E CE47 B889 DF7F FED1 389A 563C C272 A126
More information about the Openstack-docs
mailing list