[PATCH 2/2] Integrate 'pre-commit'
Daniel Axtens
dja at axtens.net
Thu Sep 26 23:12:57 AEST 2019
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?
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
More information about the Patchwork
mailing list