CODEWITHSUNDEEP
pythongitpre-commit-hookpre-commitpre-commit.com
baxx
baxx

Reputation: 4695

With pre-commit, how to use some hooks before commit and others before push

Some hooks can take a while to run, and I would like to run those before I push, but not before each particular commit (for example, pylint can be a bit slow).

I've seen the following:

But it's still not clear to be how I'm supposed to set this up.

Here is what I have tried:

default_stages: [commit]
repos:
  - repo: https://github.com/pre-commit/pre-commit-hooks
    rev: v3.1.0
    hooks:
      - id: end-of-file-fixer
      - id: trailing-whitespace
  - repo: https://github.com/psf/black
    rev: 19.10b0
    hooks:
      - id: black
        stages: [push]

From that I'm expecting the first couple of hooks to run before a commit (which they do), but I'm expecting black to run before pushing, which it doesn't.

To test that I have created the following file:

"""This is a docstring."""

print('this should be formatted')

Which is certainly not being formatted by black.

Upvotes: 24

Views: 14389

Answers (1)

anthony sottile
anthony sottile

Reputation: 69824

your configuration is correct, except that the whitespace hooks in pre-commit/pre-commit-hooks set stages themselves so they won't be affected by default_stages

adjusting your configuration slightly:

repos:
  - repo: https://github.com/pre-commit/pre-commit-hooks
    rev: v3.1.0
    hooks:
      - id: end-of-file-fixer
        stages: [commit]
      - id: trailing-whitespace
        stages: [commit]
  - repo: https://github.com/psf/black
    rev: 19.10b0
    hooks:
      - id: black
        stages: [push]

next you'll need to make sure both of the hook scripts are installed

You can install both the pre-commit and pre-push commit at the same time using:

pre-commit install --hook-type pre-commit --hook-type pre-push

or you can run them separately:

pre-commit install  # installs .git/hooks/pre-commit
pre-commit install --hook-type pre-push  # installs .git/hooks/pre-push

note that the second command comes directly from the documentation on using pre-push

disclaimer: I'm the author of pre-commit and pre-commit-hooks

Upvotes: 36

Related Questions