-
Notifications
You must be signed in to change notification settings - Fork 5
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
MAINT - Update pre-commit hooks #45
Conversation
|
Approved, but this might conflict with other so Maybe merge later as it will be easier to rebase this (and rerun the precomit hooks) than other way around. |
Good shout on the (not) merging. I added a |
f4f4e3b
to
79b07ec
Compare
Rebased / fixed conflicts by regenerating the files. |
One thing I did not do was delete the |
#41 (merged) deletes the entire test/results directory, so I'm not sure why that's not a merge conflict with main |
Co-authored-by: gabalafou <gabriel@fouasnon.com>
All looks good, let's merge, I don't think it will conflicts with #33 |
As I was reviewing some PRs I noticed our pre-commit hooks needed an update and that some files (HTML) were not being linted/formated.
Included in this PR:
pyproject.toml
There are a lot of changes as I ran the linter, so the only files needing a check are
pyproject.toml
and.pre-commit-config.yaml