Skip to content

Conversation

@julien-lang
Copy link
Contributor

Previous warning messages said removal will be March or July 2026. Postponing all removal to September 2026.

@julien-lang julien-lang requested a review from Copilot November 20, 2025 19:32
Copy link
Contributor

Copilot AI left a 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.

@julien-lang julien-lang requested a review from Copilot November 20, 2025 19:39
Copy link
Contributor

Copilot AI left a 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.

@julien-lang julien-lang marked this pull request as ready for review November 20, 2025 19:41
@julien-lang julien-lang requested a review from a team November 20, 2025 19:41
Copy link
Contributor

@carlos-villavicencio-adsk carlos-villavicencio-adsk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@julien-lang julien-lang merged commit 60630cf into master Nov 24, 2025
24 checks passed
@julien-lang julien-lang deleted the ticket/SG-40633 branch November 24, 2025 20:14
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.

4 participants