[libvirt PATCH 23/23] build: add syntax-check rules for undesirable terms

Daniel P. Berrangé berrange at redhat.com
Fri Jun 19 12:33:43 UTC 2020


On Fri, Jun 19, 2020 at 02:11:09PM +0200, Ján Tomko wrote:
> On a Friday in 2020, Peter Krempa wrote:
> > On Fri, Jun 19, 2020 at 10:33:00 +0100, Daniel Berrange wrote:
> > > We don't check for "master", because there are too many
> > > cases that we're not trying to eliminate at this time.
> > > 
> 
> Even if you consider the terms undesirable, consider using them in the
> commit summary instead of mentioning what you're not trying to do.
> 
> > > Signed-off-by: Daniel P. Berrangé <berrange at redhat.com>
> > > ---
> > >  build-aux/syntax-check.mk | 16 ++++++++++++++++
> > >  1 file changed, 16 insertions(+)
> > 
> > I don't think there's a technical reason forbiding these and it's almost
> > borderline censorship. I refuse to put my R-b on this one.
> > 
> 
> Given how many files are excepted, I think it's a waste of electricty to
> even check for these.

We have > 10,000 files in source control, of which only 150 are
exempted and time required to check that won't even register
in the noise. We shouldn't be relying on reviewers to check things
that can trivially be automated, when we know reviewers often miss
things.

Regards,
Daniel
-- 
|: https://berrange.com      -o-    https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org         -o-            https://fstop138.berrange.com :|
|: https://entangle-photo.org    -o-    https://www.instagram.com/dberrange :|




More information about the libvir-list mailing list