Remove pip pin from App Engine requirements - #13759
Conversation
|
👋 Hello @glenn-jocher, thank you for submitting a To ensure a seamless integration of your work, please review the following checklist: For more guidance, please refer to our Contributing Guide. Don't hesitate to leave a comment if you have any questions. Thank you for contributing to Ultralytics! 🚀 |
UltralyticsAssistant
left a comment
There was a problem hiding this comment.
🔍 PR Review
Made with ❤️ by Ultralytics Actions
Clean PR. Removing the explicit pip pin from the App Engine supplemental requirements is a narrow, low-risk change, and I don't see any concrete bug, compatibility, or maintainability issue in the provided diff.
|
🎉 Merged — thanks, @glenn-jocher! “Perfection is achieved, not when there is nothing more to add, but when there is nothing left to take away.” — Antoine de Saint-Exupéry This cleanup is a great example of that idea in practice: removing the pinned |
Summary
pip==26.0pipreleaseValidation
rg -n "^pip([<>=!~ ]|$)|pip==" -g "*requirements*.txt" -g "pyproject.toml" -g "setup.cfg" -g "setup.py"git diff --check/tmp/yolov5-pip-alert-venv/bin/python -m pip install --dry-run -r utils/google_app_engine/additional_requirements.txt🛠️ PR Summary
Made with ❤️ by Ultralytics Actions
🌟 Summary
🧹 This PR removes the pinned
pip==26.0dependency from the Google App Engine supplemental requirements to avoid unnecessarily forcing a specificpipversion during deployment.📊 Key Changes
pip==26.0fromutils/google_app_engine/additional_requirements.txtFlask==3.1.3gunicorn==23.0.0werkzeug>=3.0.1🎯 Purpose & Impact
pipas an application dependency ⚙️pipseparately ☁️