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

I think the pre-commit yaml isnt working #278

Open
hayesla opened this issue Nov 27, 2024 · 1 comment
Open

I think the pre-commit yaml isnt working #278

hayesla opened this issue Nov 27, 2024 · 1 comment
Labels

Comments

@hayesla
Copy link
Member

hayesla commented Nov 27, 2024

Describe the bug

using pre-commit raises errors:

(sunpy_py312) laurahayes@MacBook-Pro-2 sunpy-sphinx-theme % pre-commit run --all-files                           
An error has occurred: InvalidManifestError: 
==> File /Users/laurahayes/.cache/pre-commit/repofq8bgfac/.pre-commit-hooks.yaml
==> At Hook(id='docformatter-venv')
==> At key: language
=====> Expected one of conda, coursier, dart, docker, docker_image, dotnet, fail, golang, haskell, lua, node, perl, pygrep, python, r, ruby, rust, script, swift, system but got: 'python_venv'
Check the log at /Users/laurahayes/.cache/pre-commit/pre-commit.log

i went into yaml file and changed language to python rather than python_venv

To Reproduce

import sunpy

sunpy.map.Map(...)
etc

Screenshots

No response

System Details

No response

Installation method

No response

@hayesla hayesla added the Bug label Nov 27, 2024
@nabobalis
Copy link
Contributor

Yeah, they have not fixed it upstream in a release. For now we can move it to main

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

No branches or pull requests

2 participants