Pre-commit error

[mengerj]

Hi,
I am attemping to add tutorials of ecosystem packages to the scverse-tutorial repo. When trying to merge my pull request, the pre-commit bot gives this error:
Expected one of commit-msg, manual, post-checkout, post-commit, post-merge, post-rewrite, pre-commit, pre-merge-commit, pre-push, pre-rebase, prepare-commit-msg but got: 'Nonepre-commitNone'

regarding the .pre-commit-config.yaml file. Not sure if this a pre-commit version issue or different default_stages should be chosen or if it is something else...
#188

Thanks!

Best,
Jonatan

[grst]

Hi @mengerj,

yes, there has recently been an update to the pre-commit config files.
Can you try to run pre-commit migrate-config?

[mengerj]

running pre-commit migrate-config didn't result in any changes. It works if you manually remove the "None" parts of the names. Do you know why they were there in the first place? Included the change in my pull-request. I wanted to have a seperate pull request for that change by checking out into a new branch first, but It seems it is still included in my previous pull-request. Hope it's fine.

[grst]

Many thanks, I have no idea how the None ended up there.

[flying-sheep]

pre-commit autoupdate had an oopsie, not our fault