Skip to content

Conversation

@deathaxe
Copy link
Member

@deathaxe deathaxe commented Jul 31, 2024

Fixes #146
Fixes #150

This PR...

  1. opts-in Helium to python 3.8
  2. adjusts dependencies for python 3.8 plugin_host
    • "enum" is now built-in and no longer needed
    • "pyzmq" 26 however requires "tornado" library

This change fixes compatibility with Apple Silicon and Linux ARM64.

Note: These are least required changes. I see some Syntax Warnings in console. Maybe a bit more action needst to be taken to fully support python 3.8

This commit...

1. opts-in Helium to python 3.8
2. adjusts dependencies for python 3.8 plugin_host
   - "enum" is now built-in and no longer needed
   - "pyzmq" 26 however requires "tornado" library
@sonarqubecloud
Copy link

@cgahr
Copy link
Collaborator

cgahr commented Aug 1, 2024

Hey, thanks for your PR!

I'm not using sublime text anymore, so sadly I cannot test the PR. Does Helium still work with ST3?

@deathaxe
Copy link
Member Author

deathaxe commented Aug 1, 2024

This PR doesn't change anything for ST3.

It only marks the plugin to run on python 3.8 in ST4, primarily to use latest pyzmq library and support arm64 platforms.

@cgahr
Copy link
Collaborator

cgahr commented Aug 1, 2024

Closes #150

@deathaxe
Copy link
Member Author

deathaxe commented Aug 1, 2024

FWIW, this PR was created together with yaroslavyaroslav.

He confirmed Helium working on ST4 MacOS M1.

@cgahr cgahr self-requested a review August 5, 2024 07:26
@cgahr cgahr merged commit 0feec81 into SublimeText:master Aug 5, 2024
@deathaxe deathaxe deleted the pr/python-38 branch August 5, 2024 13:20
@deathaxe
Copy link
Member Author

deathaxe commented Aug 5, 2024

Thanks for your efforts.

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

Successfully merging this pull request may close these issues.

Not working on ST4 - Mac M1

2 participants