search

conda-forge / conda-smithy

The tool for managing conda-forge feedstocks.
https://conda-forge.org/
BSD 3-Clause "New" or "Revised" License
146 stars 169 forks source link

lint on most wheel-based python installs except `noarch: python` and `none-any.whl` #1935

Closed beckermr closed 1 month ago

beckermr commented 1 month ago

There are now several feedstocks that use pure python wheels with noarch: python recipes. This procedure is in principle safe, but an sdist is preferred. We should lint on any usage of wheels outside this one case. A second PR might also provide a hint that sdists are preferred.

beckermr commented 1 month ago

cc @jaimergp

hmaarrfk commented 1 month ago

One challenge is that many first time packagers on pypi will only publish wheels and not sdist.

Should the message be condition on the existing of an sdist for the same version of the package?

hmaarrfk commented 1 month ago

Sorry. My comment did understand that you are ok with the wheel + no arch case