-
Notifications
You must be signed in to change notification settings - Fork 0
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
Schema validation for hatch fails with [tool.hatch.env]
#31
Comments
https://github.com/SchemaStore/schemastore/blob/master/src/schemas/json/hatch.json probably needs to be fixed, then. |
@flying-sheep, was this hand written or generated somehow? |
The part that's a bit confusing is one can see the definitions in the schema: https://github.com/SchemaStore/schemastore/blob/7a7f32e81810a8909773810564fe52b43f31bcd4/src/schemas/json/hatch.json#L376-L413 with My pre-commit is:
Am I just maybe somehow pinned to an older version of the schema store? |
Ahh, I think this might not be updating. It should have changed in SchemaStore/schemastore@c20b9a0 but don't see anything in our last release. Oh, maybe it was the previous release, it does seem to be there. Ahh, let's just add your snippet as a test. |
Ahh, you are not pinning on us, so it's using an old cached version. Pre-commit globally caches. If you put You can instead put this repo as your pre-commit hook, then it pins on this instead of validate-pyproject. |
I did follow the README instructions you have in this project validate-pyproject-schema-store/README.md Lines 25 to 32 in 353a7a8
>= in additional dependencies.
|
I'd add The other way to do it would be to use repos:
- repo: https://github.com/henryiii/validate-pyproject-schema-store
rev: <insert here>
hooks:
- id: validate-pyproject Which is an alternative suggested in the README. |
So the former suggestion you mentioned doesn't work for me using
which gives
so I'll switch to using this repo as the source for the pre-commit hook. |
Ahh, yes, that’s a pre-commit quirk, svn versioning no longer works for the main repo being checked out. Maybe we should drop the min version requirement. |
In the hatch documentation, if one wants to use a different environment type, then you need to specify, for instance with juftin/hatch-pip-compile
However, the current schema definition seems to fail with
I'm not sure if this is expected, as I'm having a hard time reading the associated PEPs to understand (and digging into the code to understand what is expected).
The text was updated successfully, but these errors were encountered: