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 <step...@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@lists.ozlabs.org > > https://lists.ozlabs.org/listinfo/patchwork _______________________________________________ Patchwork mailing list Patchwork@lists.ozlabs.org https://lists.ozlabs.org/listinfo/patchwork