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

Python2 removal #1525

Draft
wants to merge 12 commits into
base: main
Choose a base branch
from
Draft

Python2 removal #1525

wants to merge 12 commits into from

Conversation

hexaltation
Copy link
Collaborator

@hexaltation hexaltation commented Mar 19, 2025

Context

We announce drop support of python 2 on may.
This PR is the removal of python 2 linked logic.

Proposed solution

Steps

Remove python 2 references from code

  • remove from Readme
  • remove from dockerfile
  • remove from buildtools
    • update generate_local_list.sh
    • clean update_schema.sh
  • app
    • remove from app/client
    • remove from app/server
      • clean pynbox references
    • remove from app/common
    • remove from app/gen-server
    • remove from app/plugin
  • remove from tests
  • remove from statics if so

Remove unused ENV VARIABLES

  • PYTHON_VERSION_ON_CREATION
  • PYTHON_VERSION
  • GRIST_SANDBOX2/3

Remove six bridges from python code

Simplify /sandbox vs sandbox_v3

  • Have unique python sandbox folder

Related issues

fixes #1441

Has this been tested?

🤞 by CI

  • 👍 yes, I added tests to the test suite
  • 💭 no, because this PR is a draft and still needs work
  • 🙅 no, because this is not relevant here
  • 🙋 no, because I need help

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.

Plan an end of support of python 2 in grist
1 participant