ignore pyzmq->auditwheel dependency in 314t migration#8272
ignore pyzmq->auditwheel dependency in 314t migration#8272h-vetinari merged 1 commit intoconda-forge:mainfrom
Conversation
auditwheel sticks on ujson, but is only used for stable-abi builds, not free-threaded
|
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 ( I do have some suggestions for making it better though... For recipe/meta.yaml:
This message was generated by GitHub Actions workflow run https://github.com/conda-forge/conda-forge-webservices/actions/runs/22285673089. Examine the logs at this URL for more detail. |
h-vetinari
left a comment
There was a problem hiding this comment.
This should also work in principle, but didn't for pyarrow, unfortunately (I must have looked up the wrong example to give you; usually I just added things to exclude:, which does need the corresponding manual PR).
.
If ignored_deps_per_node worked as intended, it wouldn't even need conda-forge/pyzmq-feedstock#126, the migrator should just open a PR despite the unmigrated dependency.
However, something doesn't seem to work with the "plucking" of nodes in the dependency graph. CC @beckermr
In any case, let's merge this; if it doesn't lead to the migrator moving on in 2-3 days, we can add pyzmq to exclude:.
auditwheel sticks on ujson, but is only used in pyzmq for tests of stable-abi builds, not free-threaded
cc @h-vetinari is this right for unblocking pyzmq?