Closed baszalmstra closed 3 months ago
Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe
) and found it was in an excellent condition.
Ouch that is a lot of errors..
I only included specific tests for now. I am unsure why a lot of tests fail but I assume its because the tests do not expect to be run from another repository?
Anyway, I think its still important to be able to run the majority of the tests for downstream tests.
@conda-forge-admin, please rerender
Hi! This is the friendly automated conda-forge-webservice.
I tried to rerender for you, but it looks like there was nothing to do.
This message was generated by GitHub actions workflow run https://github.com/conda-forge/conda-smithy-feedstock/actions/runs/10006117311.
Pydantic is not a runtime requirement afaik. Cc @isuruf who knows this better.
There is a specific pytest invocation to use for installed packages. We should try that here.
Would be happy to! Do you know what the invocation is, a quick google didnt tell me much?
Pydantic is not a runtime requirement afaik. Cc @isuruf who knows this better.
Its used here:
I think thats part of the package.
We currently serialize the schema as json and don't use pydantic at runtime IIRC.
We currently serialize the schema as json and don't use pydantic at runtime IIRC.
But then import conda_smithy.schema
would just fail right?
True but I don't think that module is ever impoted at runtime when using smithy.
See here:
https://github.com/search?q=repo%3Aconda-forge%2Fconda-smithy%20conda_smithy.schema&type=code
this is by design AFAIK. Other members of @conda-forge/core should confirm or deny.
See here:
github.com/search?q=repo:conda-forge/conda-smithy%20conda_smithy.schema&type=code
this is by design AFAIK. Other members of @conda-forge/core should confirm or deny.
I wonder why that would be by design because that seems like bad practice. I can imagine it is used just to generate the JSON schema but I dont think it belongs inside the package in that case.
But I would happy to remove it, Ill wait for someone from core to reply.
I do not want to rehash the pydantic issue since it has been an ongoing thing. I doubt this PR will get merged until it is removed.
Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe/meta.yaml
) and found it was in an excellent condition.
I do not want to rehash the pydantic issue since it has been an ongoing thing. I doubt this PR will get merged until it is removed.
Ok! I moved it to the test section instead.
Hi! This is the friendly conda-forge automerge bot!
I considered the following status checks when analyzing this PR:
Thus the PR was passing and merged! Have a great day!
Checklist
0
(if the version changed)conda-smithy
(Use the phrase code>@<space/conda-forge-admin, please rerender in a comment in this PR for automated rerendering)This PR adds running of the pytest to the recipe test section. This is useful so that we can run downstream tests from dependencies to guard against accidental breakage.
I also added
pydantic >=2,<3
to the run requirements. This is also the case in the upstreamenvironment.yaml
.