On Fri, Sep 18, 2020 at 10:16 PM Daniel P. Berrangé <email@example.com
> On Fri, Sep 18, 2020 at 04:07:22PM +0200, Thomas Huth wrote:
> > On 18/09/2020 15.29, Daniel P. Berrangé wrote:
> > > This introduces two new jobs to GitLab. The first runs "checkpatch.pl
> > > across all patches, while the second is a dedicated DCO signoff check.
> > This feels quite redundant since we're checking the patches with Patchew
> > already ... or are there plans to get rid of this check in Patchew?
> patchew only runs once the contributor has sent their patches to the
> mailing list, whci his too late.
> We want contributors to test their series in GitLab CI ahead of sending
> it, so that patchew never has to report any failure, because the code is
> already perfect once on the list (except if git master has moved and
> causes conflicts of course).
> |: https://berrange.com
> |: https://libvirt.org
> |: https://entangle-photo.org
agreed, and ineed patchew are broken now.