Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

noahp/pre commit #66

Draft
wants to merge 4 commits into
base: master
Choose a base branch
from
Draft

noahp/pre commit #66

wants to merge 4 commits into from

Commits on Dec 20, 2022

  1. Add a pre-commit config file 

    Not enabled in CI, but helpful for linting the github actions script as
well as the python code.

I haven't run this against all-files.
    @noahp
    noahp committed Dec 20, 2022
    Configuration menu
    Copy the full SHA
    c5fd70c View commit details
    Browse the repository at this point in the history

  2. Add pre-commit 

    Cover all the usual ones.

No pylint or mypy.
    @noahp
    noahp committed Dec 20, 2022
    Configuration menu
    Copy the full SHA
    d7a7bdb View commit details
    Browse the repository at this point in the history

  3. Run pre-commit and commit the result

    @noahp
    noahp committed Dec 20, 2022
    Configuration menu
    Copy the full SHA
    d4fe3f7 View commit details
    Browse the repository at this point in the history

  4. Add .git-blame-ignore-revs for pre-commit run

    @noahp
    noahp committed Dec 20, 2022
    Configuration menu
    Copy the full SHA
    1cbf1e2 View commit details
    Browse the repository at this point in the history