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

fix: pre-commit configuration filename #32

Merged

Conversation

adamantike
Copy link
Contributor

According to pre-commit official documentation [0]:

A git repo containing pre-commit plugins must contain a .pre-commit-hooks.yaml file that tells pre-commit:

With the current filename (yml instead of yaml), trying to configure this project as a hook makes pre-commit fail with:

An error has occurred: InvalidManifestError:
=====> .local/cache/pre-commit/repog7znoma3/.pre-commit-hooks.yaml is not a file

[0] https://pre-commit.com/#creating-new-hooks

According to `pre-commit` official documentation [0]:

> A git repo containing pre-commit plugins must contain a .pre-commit-hooks.yaml file that tells pre-commit:

With the current filename (`yml` instead of `yaml`), trying to configure
this project as a hook makes `pre-commit` fail with:

```
An error has occurred: InvalidManifestError:
=====> .local/cache/pre-commit/repog7znoma3/.pre-commit-hooks.yaml is not a file
```

[0] https://pre-commit.com/#creating-new-hooks
@tonybaloney tonybaloney merged commit f02e0f3 into tonybaloney:main Jun 21, 2022
@tonybaloney
Copy link
Owner

Thanks!!

@adamantike adamantike deleted the fix/pre-commit-yaml-filename branch September 18, 2022 04:40
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 this pull request may close these issues.

2 participants