Skip to content

Conversation

@julien-lang
Copy link
Contributor

@julien-lang julien-lang commented Nov 7, 2025

Related to shotgunsoftware/tk-core#1069

Follow up from #180

New banner:

image image

@julien-lang julien-lang requested a review from Copilot November 7, 2025 18:35
Copy link

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 updates the Python 3.7 end-of-life notification banner to announce upcoming breaking changes. The notification ID is changed to reflect that this is now about breaking changes rather than just deprecation, and the messaging is updated to reflect new timelines and version requirements (March 2026 deadline, Python 3.11 recommended).

Key changes:

  • Updated notification ID from "deprecation-notification-python37" to "python37-breaking-changes"
  • Changed messaging from February 2025 deadline to March 2026 with breaking changes emphasis
  • Updated recommended Python version from 3.9+ to 3.11

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link

@mchesnay mchesnay left a comment

Choose a reason for hiding this comment

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

Comments to address before merging. As stated in Slack, I'm not comfortable being the sole reviewer on this: a PM should be in the loop and approve the text.
Also, could the date be March 31 or April 2? That change happening on April 1 (April's fool day) could be an issue

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 requested a review from Copilot November 13, 2025 23:37
Copy link

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 1 out of 1 changed files in this pull request and generated 2 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@julien-lang julien-lang requested a review from Copilot November 14, 2025 16:21
Copy link

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 1 out of 1 changed files in this pull request and generated 1 comment.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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.

5 participants