Skip to content
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

Isort misses bundled libs #711

Closed
apollo13 opened this issue Oct 6, 2023 · 7 comments
Closed

Isort misses bundled libs #711

apollo13 opened this issue Oct 6, 2023 · 7 comments

Comments

@apollo13
Copy link
Contributor

apollo13 commented Oct 6, 2023

The https://open-vsx.org/extension/ms-python/isort extension misses the bundled libraries. @filiptronicek (I hope it's okay to ping you for such things) is there a way to find out which action run published the extension to see whether or not there have been errors.

Also since isort as of now still uses python 3.7 we could try if the publishing workflow can install python3.7 and python3.8 side-by-side. But at the end of the day we would require to republish isort -- is that somehow possible?

@apollo13
Copy link
Contributor Author

apollo13 commented Oct 6, 2023

@apollo13
Copy link
Contributor Author

apollo13 commented Oct 6, 2023

Attached is a build from my PC that works -- though I fully understand if you don't want to pull that in. It is built from tag v2023.11.12711013 isort.zip

@filiptronicek
Copy link
Member

filiptronicek commented Oct 6, 2023

Very well, I built the extension incorrectly on Python 3.11 instead of 3.7. Thanks, @apollo13, for looking into it!

image

I have now pushed a hack for the extension to use 3.7 when publishing, but will introduce an optional pythonVersion property for extensions sometime next week. Will also look into unpublishing the current one so that it can be replaced.

@apollo13
Copy link
Contributor Author

apollo13 commented Oct 6, 2023

Thank you for your hard work. pythonVersion sounds like a good idea. I wonder though if it might be easier to just call the setup python action twice? I think the second run will leave the first python version around and mostly replace the python symlink. But I'll leave it up to you to decide what you think would be better (maybe performance/timings of the action run should be a consideration?)

@filiptronicek
Copy link
Member

@apollo13 I have republished ms-python.isort v2023.10.1. Could you check it out and see if it okay now? 🙏

@apollo13
Copy link
Contributor Author

apollo13 commented Oct 6, 2023

@filiptronicek Could there be a caching issue, or does keep vscode the downloaded files locally? If I manually download the vsix it has /extension/bundled/libs/ -- but if I install it via codium it doesn't have it.

@apollo13
Copy link
Contributor Author

apollo13 commented Oct 6, 2023

Ah yes, .config/VSCodium/CachedExtensionVSIXs. After uninstalling the extension, stopping code and removing the cached vsix I could properly reinstall. Thx!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants