Skip to content

fix: add contents:write permission to release workflow#331

Merged
MaorEJFrog merged 1 commit intomasterfrom
feature/JA-20235/fix-release-workflow-permissions
Mar 22, 2026
Merged

fix: add contents:write permission to release workflow#331
MaorEJFrog merged 1 commit intomasterfrom
feature/JA-20235/fix-release-workflow-permissions

Conversation

@MaorEJFrog
Copy link
Collaborator

@MaorEJFrog MaorEJFrog commented Mar 22, 2026

The release workflow needs explicit contents:write permission to force-push the v4 major version tag. Without this permission, the v4 tag is not updated when new releases are published, causing users calling @v4 to get outdated versions.

This fixes the issue where v4.10.0 was released but the v4 tag was not updated, leaving users on @v4 stuck with Node 20 instead of Node 24.

Relates to: JA-20235

  • All tests passed. If this feature is not already covered by the tests, I added new tests.
  • I used npm run format for formatting the code before submitting the pull request.

Related to #306 (comment)

The release workflow needs explicit contents:write permission to force-push
the v4 major version tag. Without this permission, the v4 tag is not updated
when new releases are published, causing users calling @v4 to get outdated
versions.

This fixes the issue where v4.10.0 was released but the v4 tag was not updated,
leaving users on @v4 stuck with Node 20 instead of Node 24.

Relates to: JA-20235

Made-with: Cursor
@github-actions
Copy link

👍 Frogbot scanned this pull request and did not find any new security issues.


@roeioshry roeioshry self-requested a review March 22, 2026 10:34
@MaorEJFrog MaorEJFrog merged commit b14f2d4 into master Mar 22, 2026
20 checks passed
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