Skip to content

Conversation

@kdmccormick
Copy link
Member

@kdmccormick kdmccormick commented Jun 12, 2025

@kdmccormick kdmccormick force-pushed the kdmccormick/fast-track-depr branch from ff82866 to 3bbcdf7 Compare June 12, 2025 16:09
@kdmccormick kdmccormick marked this pull request as ready for review June 12, 2025 16:15
@robrap
Copy link
Contributor

robrap commented Jun 12, 2025

@kdmccormick:

  1. Do you want to provide a PR description here?
  2. Should we create a Supporting Deprecation issue for this, so we can track other follow-up. Possibilities include retro of this if we consider it a POC/experiment, and/or updating the OEP to document the decision (which could be documented in this PR description or the issue description).

@@ -0,0 +1,40 @@
name: 🚅 Fast-Track Deprecation/Removal (DEPR)
description: Per OEP-21, use this template when you are making a pre-approved breaking change. These include: Pre-planned upgrade tasks, removal of unreachable code, and subtasks of other approved DEPR tickets. Consult the DEPR Working Group if unsure.
Copy link
Contributor

Choose a reason for hiding this comment

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

See #174 (comment) for request for an issue on this topic, until OEP-21 can be updated. The issue can refer to OEP-21. The new issue can be inserted into the text below in some nicely readable format.

Suggested change
description: Per OEP-21, use this template when you are making a pre-approved breaking change. These include: Pre-planned upgrade tasks, removal of unreachable code, and subtasks of other approved DEPR tickets. Consult the DEPR Working Group if unsure.
description: Use this template when you are making a pre-approved breaking change. These include: Pre-planned upgrade tasks, removal of unreachable code, and subtasks of other approved DEPR tickets. Consult the DEPR Working Group if unsure. See [Fast/Track DEPR proof-of-concept ticket](<insert issue>) for details.

@@ -1,4 +1,4 @@
name: 🌘 Deprecation and Removal (DEPR) Ticket
Copy link
Contributor

Choose a reason for hiding this comment

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

Note: The term "ticket" is used in the OEP. Is there an expected follow-up to change that?

Copy link
Member Author

Choose a reason for hiding this comment

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

Can do

kdmccormick and others added 2 commits June 12, 2025 14:04
Co-authored-by: Taylor Payne <taylor.payne@wgu.edu>
@kdmccormick kdmccormick force-pushed the kdmccormick/fast-track-depr branch from 54e0a31 to 126df46 Compare June 12, 2025 18:04
Co-authored-by: Robert Raposa <rraposa@edx.org>
@kdmccormick kdmccormick mentioned this pull request Jun 12, 2025
6 tasks
@kdmccormick
Copy link
Member Author

Comments addressed. Please LMK if you have any more blocking suggestions.

I'm looking to get this merged soon so I can start using it, but I do plan to open another PR to clean up the language on both templates a bit. Happy to take nit-picks over there.

@kdmccormick kdmccormick merged commit 6e6dc58 into master Jun 12, 2025
1 check passed
@kdmccormick kdmccormick deleted the kdmccormick/fast-track-depr branch June 12, 2025 19:35
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