Skip to content

pre-commit install instructions are incomplete #1360

Description

@encukou

The current pre-commit install instructions are listed before the “Compile and build” section:

$ pre-commit install --allow-missing-config
pre-commit installed at .git/hooks/pre-commit

This doesn't work before pre-commit itself is installed, and it can't be installed if one doesn't have Python yet.

Activity

  1. hugovk commented on Aug 15, 2024

    @hugovk
    Member

    The instructions in full are:


    Install pre-commit as a Git hook

    To make sure your code is linted correctly, we recommend setting up pre-commit as a Git hook:

    $ pre-commit install --allow-missing-config
    pre-commit installed at .git/hooks/pre-commit
    

    Now pre-commit will run automatically on git commit.


    That pre-commit link is to https://pre-commit.com/#installation which shows a couple of ways to install pre-commit.

    I don't think we need to instruct people how to install Python itself: I think if someone wants to contribute to Python development, they will already have a Python installation, or know how to do so. And if nothing else, the front page of the devguide begins with instructions on how to build Python.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions