Skip to content

Bump nodejs22 to 22.5.0#1626

Closed
omBratteng wants to merge 1 commit into
GoogleContainerTools:mainfrom
omBratteng:update-node-22
Closed

Bump nodejs22 to 22.5.0#1626
omBratteng wants to merge 1 commit into
GoogleContainerTools:mainfrom
omBratteng:update-node-22

Conversation

@omBratteng

Copy link
Copy Markdown
Collaborator

No description provided.

@omBratteng

Copy link
Copy Markdown
Collaborator Author

@loosebazooka the report diff keeps failing on PRs made by contributors outside the GoogleContainerTools org. If you update the workflow so that it has write permissions to pull_request, it should no longer fail for external contributors.

@smorimoto

Copy link
Copy Markdown
Contributor

Note that you will have to wait a while to merge this change: nodejs/node#53934

@omBratteng

Copy link
Copy Markdown
Collaborator Author

I guess we probably can skip this version, and wait for a patched version, 22.5.1 probably.

@loosebazooka

Copy link
Copy Markdown
Member

@loosebazooka the report diff keeps failing on PRs made by contributors outside the GoogleContainerTools org. If you update the workflow so that it has write permissions to pull_request, it should no longer fail for external contributors.

Yeah this was more to detect issues on large scale changes we were doing

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.

3 participants