-
Notifications
You must be signed in to change notification settings - Fork 925
Fix wheel extension in verify_checksums.sh #2092
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
Conversation
tisonkun
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you!
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## trunk #2092 +/- ##
=======================================
Coverage 83.34% 83.34%
=======================================
Files 353 353
Lines 81769 81769
Branches 8993 8993
=======================================
Hits 68148 68148
Misses 10796 10796
Partials 2825 2825 🚀 New features to boost your workflow:
|
|
I detected another issue getting the wheel from the pypi repo. Also fixed now. |
|
Hi @PsiACE and @tisonkun, this change: 0a37446#diff-b4c2a69650f9ac84008ad9f745859a645c9466350ffdfe5f919655039ee2e2c3L51 has set an incorrect name in the tests, so I cannot merge this change (where I revert this change). How could we fix this? |
|
I can we can drop python version name in the status. It's meaningless as well. Let me try to prepare a fix. |
|
I alredy make a fix |
Fix wheel extension in verify_checksums.sh
Description
Verify Artifacts Checksum are failing due to an incorrect wheel file name, it uses
-py3-none-any.whlextension instead of the current-py2.py3-none-any.whl(see: https://downloads.apache.org/libcloud/)Status
Checklist (tick everything that applies)