Skip to content

Comments

ci(bindings/node): Add contents permissions for nodejs release#6182

Merged
Xuanwo merged 1 commit intomainfrom
fix-nodejs-binding
May 14, 2025
Merged

ci(bindings/node): Add contents permissions for nodejs release#6182
Xuanwo merged 1 commit intomainfrom
fix-nodejs-binding

Conversation

@Xuanwo
Copy link
Member

@Xuanwo Xuanwo commented May 14, 2025

Which issue does this PR close?

Closes #5957

Rationale for this change

Nodejs release seem require new permissions.

What changes are included in this PR?

Are there any user-facing changes?

@dosubot dosubot bot added size:XS This PR changes 0-9 lines, ignoring generated files. releases-note/ci The PR modifies CI-related content or has a title that begins with "ci" labels May 14, 2025
Copy link
Member

@dqhl76 dqhl76 left a comment

Choose a reason for hiding this comment

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

Thanks

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label May 14, 2025
@Xuanwo Xuanwo merged commit 92400d8 into main May 14, 2025
18 checks passed
@Xuanwo Xuanwo deleted the fix-nodejs-binding branch May 14, 2025 08:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm This PR has been approved by a maintainer releases-note/ci The PR modifies CI-related content or has a title that begins with "ci" size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Nodejs Binding failed to release

3 participants