-
Notifications
You must be signed in to change notification settings - Fork 31
feat: Introduce Fast-Track DEPR Template #174
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
ff82866 to
3bbcdf7
Compare
|
| @@ -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. | |||
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.
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.
| 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 | |||
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.
Note: The term "ticket" is used in the OEP. Is there an expected follow-up to change that?
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.
Can do
Co-authored-by: Taylor Payne <taylor.payne@wgu.edu>
54e0a31 to
126df46
Compare
Co-authored-by: Robert Raposa <rraposa@edx.org>
|
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. |
Adds a Fast-Track DEPR issue template as described in #175
https://github.com/openedx/.github/blob/kdmccormick/fast-track-depr/.github/ISSUE_TEMPLATE/depr-ticket-fast-track.yml