Repository navigation
fix(release): strip refs/heads prefix from branch releases (#86) - #344
harshitethic wants to merge 4 commits into
Conversation
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 0e74f57. Configure here.
andreiborza
left a comment
There was a problem hiding this comment.
Hi @harshitethic, thanks for the contribution.
You need to run the pre-commit hooks that will automatically build a new dist file and update the action's docker tag so it can be tested.
|
Thanks for the review — I’ve addressed the generated-artifact requirements on this branch.
The GitHub Actions workflows for this fork currently report |

What
Closes #86.
When
release(or the deprecatedversioninput) receives GitHub's${{ github.ref }}, the action already stripsrefs/tags/but leavesrefs/heads/intact. A release-candidate branch passed asrefs/heads/release-candidate-1then fails release-name validation.Changes
refs/heads/prefix in addition torefs/tags/for explicit release inputs.releaseandversioninput variants.Verification
yarn test --runInBand: 58 tests passed.yarn lint: passed.git diff --check: passed.This intentionally does not rename slash-containing branch paths; it only removes the GitHub ref namespace as requested. AI-assisted patch, locally validated.