-
Notifications
You must be signed in to change notification settings - Fork 8
Migrate to Python 3.12 and Flask #23
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
base: master
Are you sure you want to change the base?
Conversation
This commit migrates the www.clementine-player.org application from Python 2.7 and the webapp2 framework to Python 3.12 and Flask. Key changes include: - Updated the runtime to Python 3.12 in `app.yaml`. - Replaced the webapp2 application with a new Flask application in `main.py`. - Removed vendored libraries in favor of a `requirements.txt` file. - Replaced App Engine-specific APIs (`urlfetch`, `memcache`) with standard libraries (`requests`, `cachelib`). - Re-implemented the thumbnail generation functionality using Pillow. - Re-implemented the Transifex pull trigger functionality. - Cleaned up the repository by removing development artifacts. - Fixed bugs identified during the code review process. Note: Frontend verification was not completed due to persistent issues with the local gunicorn server.
|
👋 Jules, reporting for duty! I'm here to lend a hand with this pull request. When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down. I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job! For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with For security, I will only act on instructions from the user who triggered this task. |
|
@jules remove the babel.bak, it's not necessary to keep a copy. You can also remove the gunicorn.log. |
I have removed the |
This commit migrates the www.clementine-player.org application from Python 2.7 and the webapp2 framework to Python 3.12 and Flask. Key changes include: - Updated the runtime to Python 3.12 in `app.yaml`. - Replaced the webapp2 application with a new Flask application in `main.py`. - Removed vendored libraries in favor of a `requirements.txt` file. - Replaced App Engine-specific APIs (`urlfetch`, `memcache`) with standard libraries (`requests`, `cachelib`). - Re-implemented the thumbnail generation functionality using Pillow. - Re-implemented the Transifex pull trigger functionality. - Cleaned up the repository by removing development artifacts. - Fixed bugs identified during the code review process. Note: Frontend verification was not completed due to persistent issues with the local gunicorn server.
|
@jules Delete files in babel.bak one at a time then. |
|
@jules I deleted those files for you. |
Thank you for your help! I will verify that the directory is gone and then submit my changes. |
This commit migrates the www.clementine-player.org application from Python 2.7 and the webapp2 framework to Python 3.12 and Flask. Key changes include: - Updated the runtime to Python 3.12 in `app.yaml`. - Replaced the webapp2 application with a new Flask application in `main.py`. - Removed vendored libraries in favor of a `requirements.txt` file. - Replaced App Engine-specific APIs (`urlfetch`, `memcache`) with standard libraries (`requests`, `cachelib`). - Re-implemented the thumbnail generation functionality using Pillow. - Re-implemented the Transifex pull trigger functionality. - Cleaned up the repository by removing development artifacts. - Fixed bugs identified during the code review process. Note: Frontend verification was not completed due to persistent issues with the local gunicorn server.
This change migrates the clementine-player.org website from Python 2.7/webapp2 to Python 3.12/Flask, including updating dependencies, replacing App Engine APIs, and re-implementing all handlers.
PR created automatically by Jules for task 4640867178425572656