[PATCH 2/2] Integrate 'pre-commit'

[Stephen Finucane]

On Thu, 2019-09-26 at 23:12 +1000, Daniel Axtens wrote:
> Hi Stephen,
> 
> I love automated tools, but I have no idea what pre-commit is - would
> you be able to clarify it in the commit message?

Yup, sorry. Let me send a follow-up doc fix that should explain
everything.

Stephen

> Kind regards,
> Daniel
> 
> > Helps me avoid simple style issues. We explicitly disable newline checks
> > for some files where the "fix" would be harmful (e.g. mboxes and
> > templates).
> > 
> > Signed-off-by: Stephen Finucane <stephen at that.guru>
> > ---
> >  .pre-commit-config.yaml | 20 ++++++++++++++++++++
> >  1 file changed, 20 insertions(+)
> >  create mode 100644 .pre-commit-config.yaml
> > 
> > diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml
> > new file mode 100644
> > index 00000000..bea1187a
> > --- /dev/null
> > +++ b/.pre-commit-config.yaml
> > @@ -0,0 +1,20 @@
> > +repos:
> > +- repo: https://github.com/pre-commit/pre-commit-hooks
> > +  rev: v2.3.0
> > +  hooks:
> > +  - id: check-executables-have-shebangs
> > +  - id: check-merge-conflict
> > +  - id: check-yaml
> > +  - id: end-of-file-fixer
> > +    exclude: (.*\.mbox)|(.*\.css)|(.*\.json)|(.*\.js)|(.*\.svg)|(.*/mails/.*-subject\.txt)
> > +  - id: flake8
> > +    exclude: ^patchwork/migrations
> > +  - id: trailing-whitespace
> > +    exclude: (.*\.mbox)|(.*\.svg)
> > +- repo: https://github.com/Lucas-C/pre-commit-hooks
> > +  rev: v1.1.7
> > +  hooks:
> > +  - id: remove-tabs
> > +    exclude: (.*\.mbox)|(.*\.svg)|(.*\.sql)|(.*\.conf)
> > +  - id: remove-crlf
> > +    exclude: (.*\.mbox)|(.*\.svg)
> > -- 
> > 2.21.0
> > 
> > _______________________________________________
> > Patchwork mailing list
> > Patchwork at lists.ozlabs.org
> > https://lists.ozlabs.org/listinfo/patchwork