[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