-
Notifications
You must be signed in to change notification settings - Fork 29
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
Tolerate distributions with no Name #368
Conversation
https://bugs.debian.org/1090247 shows a napari test failure caused by this on Python 3.13. On closer inspection I found that `/usr/lib/python3.13/dist-packages/coverage-7.6.0.egg-info` and `/usr/lib/python3.13/dist-packages/tqdm-4.67.1.dist-info` existed with no `Name` metadata field. This shouldn't cause a failure here.
Thanks for the PR @cjwatson! You've caught us at an awkward time because npe2api relied on web scraping PyPI search results for some functionality, and that was recently broken by PyPI due to abuse. I think this is why the CI is failing. We're still figuring out the best fix but it'll probably be minimum 1-2 weeks until a fix because of the Christmas/New Year's break. 🙏 |
For other devs stumbling on this, this is the relevant bit of the report:
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #368 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 37 37
Lines 2709 2709
=========================================
Hits 2709 2709 ☔ View full report in Codecov by Sentry. |
OK the reason we would ever be checking the |
https://bugs.debian.org/1090247 shows a napari test failure caused by this on Python 3.13. On closer inspection I found that
/usr/lib/python3.13/dist-packages/coverage-7.6.0.egg-info
and/usr/lib/python3.13/dist-packages/tqdm-4.67.1.dist-info
existed with noName
metadata field. This shouldn't cause a failure here.