-
-
Notifications
You must be signed in to change notification settings - Fork 2.9k
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
misc/upload-pypi.py errors when uploading wheel for platform emscripten_3_1_14_wasm32 #14026
Comments
I'll get more time to look into this tomorrow. Just wanted to open this in case there's anything obvious I was missing. |
I think PyPI just doesn't support wasm wheels. We can simply skip this wheel I suppose. |
cc @ethanhs |
Yup, confirming fine to skip if it unblocks the release; we just started building WASM wheels recently |
Looks like we're missing some other wheels that would be expected though: https://pypi.org/project/mypy/0.990/#files (e.g. there is only one 3.10 wheel, the previous release had 6). I'd recommend patching |
Yeah, I saw we added wasm wheel in 11be378 |
Yes, sorry, I didn't think about this, Emscripten doesn't currently have a stable ABI, so you can't upload to PyPI yet. Skipping the upload of the WASM wheel is definitely the correct thing to do here. |
Ok, I locally hacked |
Tried to make it in a way such that we can add more rules for platforms we want to filter out in the future. Tested it with `python3 misc/upload-pypi.py --dry-run 0.990` Fixes #14026
Tried to make it in a way such that we can add more rules for platforms we want to filter out in the future. Tested it with `python3 misc/upload-pypi.py --dry-run 0.990` Fixes #14026
When running
python3 misc/upload-pypi.py 0.990
there was an error when uploading wheel for platform emscripten_3_1_14_wasm32The text was updated successfully, but these errors were encountered: