Skip to content

ref: Remove #retract command#7120

Merged
BYK merged 1 commit intomainfrom
remove/retract-command
Feb 10, 2026
Merged

ref: Remove #retract command#7120
BYK merged 1 commit intomainfrom
remove/retract-command

Conversation

@BYK
Copy link
Member

@BYK BYK commented Feb 10, 2026

Summary

  • Remove the #retract comment command — engineers now have direct access to close publish issues themselves, making this feature redundant.

Changes

Deleted:

  • .github/workflows/retract.yml — workflow that listened for #retract comments
  • src/modules/retract-release.js — core retract logic (post comment + close issue)
  • src/publish/retract.js — GitHub Actions entry point
  • src/modules/__tests__/retract-release.js — tests

Edited:

  • README.md — removed the #retract flowchart branch and the "Retracting Release Request" section
  • src/modules/__tests__/update-issue.js — removed #retract instruction from test fixture strings

Engineers now have access to close publish issues directly, making the
#retract comment command redundant. This removes the workflow, module,
entry point, tests, and documentation for the feature.
@BYK BYK marked this pull request as ready for review February 10, 2026 22:08
@BYK BYK requested a review from a team as a code owner February 10, 2026 22:08
@BYK BYK merged commit 89f6457 into main Feb 10, 2026
7 checks passed
@BYK BYK deleted the remove/retract-command branch February 10, 2026 22:08
@christophaigner
Copy link
Member

@BYK FYI we still have this in our issue template (example):

To retract the release, the person requesting it must leave a comment containing #retract on a line by itself under this issue.

Also, this was actually useful for contractors, because they cannot retract issues (no permissions to close issues). How should this be handled? :)
Thanks!

@BYK
Copy link
Member Author

BYK commented Feb 23, 2026

@christophaigner you still have it in your issue template as you have not updated your create release action 😀

I'm fine bringing it back if it adds value, have not thought about contractor case.

@christophaigner
Copy link
Member

@BYK ah, well that makes sense 😄

I think it'd be great to bring it back, that's probably the easiest way to handle this scenario :)

Thank you! 🙏🏻

@christophaigner
Copy link
Member

christophaigner commented Feb 25, 2026

@BYK is it just this PR that needs to be reverted? Could you please revert it? :)

christophaigner added a commit that referenced this pull request Feb 27, 2026
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.

2 participants