-
Notifications
You must be signed in to change notification settings - Fork 119
SG-40633 Postpone removal to September 2026 #1070
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR postpones the deprecation removal timeline for several modules from March/July 2026 to September 2026. The changes update deprecation warning messages across multiple files to reflect the new timeline and also include minor text improvements (changing "FPTR" to "FPT" and refining some wording).
Key Changes
- Updated deprecation warning dates from "March 2026" or "2026-07" to "September 2026"
- Standardized product name from "FPTR" to "FPT" in warning messages
- Refactored duplicate deprecation message strings into a single variable in
__init__.py
Reviewed Changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
| python/tank_vendor/six.py | Updated deprecation warning to September 2026 and changed product name to "FPT" |
| python/tank_vendor/sgutils.py | Updated all deprecation warnings (module-level and method-level) to September 2026 |
| python/tank/authentication/sso_saml2/core/init.py | Updated deprecation date to September 2026 and refactored message into a variable |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
carlos-villavicencio-adsk
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Previous warning messages said removal will be March or July 2026. Postponing all removal to September 2026.