Conversation
Add a project override for Python Mode for Processing 4 warning users that it is unmaintained and incompatible with current Processing 4 releases. Point users to py5 as an alternative Python option.
✅ Deploy Preview for processing-contributions ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Context
Python Mode for Processing 4 was never fully completed. It is not being tested with new releases of Processing 4, and the little compatibility it had on release has gotten worse over time.
The Python Mode repo README says:
Despite this, Python Mode for Processing 4 appears in the Contribution Manager alongside the Processing 3 version, with no warnings or indication that it has been deprecated.
This was before my time so I'm not entirely sure why Python Mode for Processing 4 was released on the Contribution Manager in the first place, but at the very least, it should be labeled in a way that makes its status clear.
Proposed fix
This PR add a project override for Python Mode for Processing 4 warning users that it is unmaintained and incompatible with current Processing 4 releases. It also point users to py5 as an alternative Python option that is being maintained.
Alternative options explored
This could also be solved by editing the original
PythonMode.txtwhich is hosted on theus-southeast-backup-elleLinode VPS. However, with all the infrastructure migrations underway, the override feels like a more robust solution.Additional notes
The upstream
PythonMode.txtcurrently returns 404 which @limzykenneth and @catilac are investigating. It is likely related to migrating the processing.org DNS to Cloudflare.Using an override ensures the deprecation notice is preserved, even when the metadata gets overwritten by
scripts/fetch_updates.pyafter the 404 is resolved.We’re also planning to add a notice on the py.processing.org website, but it’s not super trivial due to how the website is deployed (it's built on old tech that isn't supported by modern CI/CD systems and rebuilds all the reference images by default on every build).