Skip to content

Conversation

@aduh95
Copy link
Contributor

@aduh95 aduh95 commented Dec 12, 2025

It was bothering me that we were logging out the BOT_TOKEN, the solution was simply to move set -ex after the preliminary checks. We were also not checking for the presence of git-node in the dependency checks, and not explicitly listing the missing deps.

@nodejs-github-bot
Copy link
Collaborator

Review requested:

  • @nodejs/actions

@nodejs-github-bot nodejs-github-bot added the tools Issues and PRs related to the tools directory. label Dec 12, 2025
@aduh95 aduh95 added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Dec 12, 2025
@aduh95 aduh95 added the commit-queue Add this label to land a pull request using GitHub Actions. label Dec 12, 2025
@nodejs-github-bot nodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Dec 14, 2025
@nodejs-github-bot nodejs-github-bot merged commit 6322071 into nodejs:main Dec 14, 2025
28 checks passed
@nodejs-github-bot
Copy link
Collaborator

Landed in 6322071

@aduh95 aduh95 deleted the create-release-proposal-improv branch December 14, 2025 18:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author ready PRs that have at least one approval, no pending requests for changes, and a CI started. tools Issues and PRs related to the tools directory.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants