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.

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

Pre-commit failing when run against all files #72

Open
proudfeet opened this issue Sep 7, 2023 · 0 comments · May be fixed by #73
Open

Pre-commit failing when run against all files #72

proudfeet opened this issue Sep 7, 2023 · 0 comments · May be fixed by #73

Comments

@proudfeet
Copy link

Hey!

(Can't see an issue template so apologies if this isn't the format you would prefer).

When experimenting locally I noticed that some of the pre-commit hooks are failing when run against all files.
Nothing major at all, just thought I would flag them so you get greens across the board on your checks.

Environment

  • OS: Ubuntu 20.04
  • Python version: 3.10.5 (through pyenv as per developer setup)

Description of the problem

Running pre-commit (as per command below) against the develop branch, across all files, results in failures.

Command:

pre-commit run --all-files

Failures are for the typo and mypy hooks.

typo
The failures here appear to be a simple config issue. Which is being tracked here.

A temporary solution is to add pass_filenames: false to the hook.

mypy
The failure here appears to be due to some historic type ignore comments on the requests_html lib. I assume they have added typing (very nice of them) at some point since you started using pre-commit.

Solution

Happy to put in PR with these small changes.

(Also noticed some of the pre-commit hooks could do with updating, I will put that in as a separate issue and PR to keep things clean).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant