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

add deprecation notice to py2wasm #53

Closed
wants to merge 1 commit into from

Conversation

karthik2804
Copy link
Contributor

@karthik2804 karthik2804 commented Feb 29, 2024

Plugin 'py2wasm' was installed successfully!

Description:
        A plugin to convert Python applications to Spin compatible modules. Note: This plugin has been deprecated in favour of the newer SDK based on componentize-py. To move to the last SDK, uninstall the plugin and visit https://developer.fermyon.com/spin/v2/python-components  

Copy link
Contributor

@itowlson itowlson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am slightly wary of the unconditional "move to componentize-py". As I understand it, this works only for Spin 2.2 or above.

A possible alternative approach would be to release a new py2wasm package, with the same binaries as the current one, but marked with ">=2.2" compatibility, and put the deprecation warning on that. Then people who are able to use the new SDK would be advised to, and people who are not would not be given advice that would break them. Thoughts?

@@ -1,6 +1,6 @@
{
"name": "py2wasm",
"description": "A plugin to convert Python applications to Spin compatible modules",
"description": "A plugin to convert Python applications to Spin compatible modules. Note: This plugin has been deprecated in favour of the newer SDK based on componentize-py. To move to the last SDK, uninstall the plugin and visit https://developer.fermyon.com/spin/v2/python-components ",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
"description": "A plugin to convert Python applications to Spin compatible modules. Note: This plugin has been deprecated in favour of the newer SDK based on componentize-py. To move to the last SDK, uninstall the plugin and visit https://developer.fermyon.com/spin/v2/python-components ",
"description": "A plugin to convert Python applications to Spin compatible modules. Note: This plugin has been deprecated in favour of the newer SDK based on componentize-py. To move to the latest SDK, uninstall the plugin and visit https://developer.fermyon.com/spin/v2/python-components.",

@@ -1,6 +1,6 @@
{
"name": "py2wasm",
"description": "A plugin to convert Python applications to Spin compatible modules",
"description": "A plugin to convert Python applications to Spin compatible modules. Note: This plugin has been deprecated in favour of the newer SDK based on componentize-py. To move to the last SDK, uninstall the plugin and visit https://developer.fermyon.com/spin/v2/python-components ",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same typo fixes throughout. But really, is there much value in retrofitting to old versions? Is anyone going to install / update to these old versions at this point? The only scenario I can think of is if someone is on Spin 0.9 or whatever and the more recent version is incompatible, but those users won't be able to run componentize-py based components anyway will they?

@karthik2804
Copy link
Contributor Author

Oh, I like the idea of bumping the version and Spin compat version. I will do that instead.

@dicej
Copy link
Contributor

dicej commented Mar 1, 2024

Oh, I like the idea of bumping the version and Spin compat version. I will do that instead.

So can we close this PR?

@karthik2804
Copy link
Contributor Author

We can close this PR. But now I'm curious while releasing a new version we want to update the binary itself to have a deprecation notice or is that too much?

@dicej
Copy link
Contributor

dicej commented Mar 1, 2024

Seems reasonable to add a deprecation notice just so they know it's no longer being supported. We don't need to push them hard to upgrade, though -- if py2wasm works fine for them they can keep using it.

@karthik2804 karthik2804 deleted the deprecate_py2wasm branch August 14, 2024 10:41
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.

3 participants