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@xxxxxxxxxx> > > > --- > > > 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 :|
