Skip to content

tools: make env variables consistent in cron jobs - #65168

Open
aduh95 wants to merge 1 commit into
nodejs:mainfrom
aduh95:consistent-cron-envs
Open

tools: make env variables consistent in cron jobs#65168
aduh95 wants to merge 1 commit into
nodejs:mainfrom
aduh95:consistent-cron-envs

Conversation

@aduh95

@aduh95 aduh95 commented Aug 9, 2026

Copy link
Copy Markdown
Contributor
  • Pass -R/--repo explicitly instead of an implicit use of GH_REPO or let the tool figure out based on the local repo
  • Prefer GH_TOKEN, since that's what gh checks first according to https://cli.github.com/manual/gh_help_environment
  • Prefer github.token over secrets.GITHUB_TOKEN to help avoid confusing it with secrets.GH_USER_TOKEN=
  • Document secrets used in auto-start-ci.yml, remove references to secrets.GH_USER_TOKEN
  • Do not pass owner and repository as argv for commit-queue.sh, it already expects to find e.g. GITHUB_SERVER_URL in the env, it's simpler for it to also expect GITHUB_REPOSITORY to be defined.
  • Move git config call to the workflow so a human running the script locally doesn't accidentally impersonate the bot.

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Review requested:

  • @nodejs/actions

@nodejs-github-bot nodejs-github-bot added meta Issues and PRs related to the general management of the project. tools Issues and PRs related to the tools directory. labels Aug 9, 2026
@aduh95
aduh95 force-pushed the consistent-cron-envs branch 4 times, most recently from 98fb3c3 to 6ef5a74 Compare August 10, 2026 13:40
Signed-off-by: Antoine du Hamel <duhamelantoine1995@gmail.com>
@aduh95
aduh95 force-pushed the consistent-cron-envs branch from 6ef5a74 to 3fcf56f Compare August 10, 2026 13:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

meta Issues and PRs related to the general management of the project. tools Issues and PRs related to the tools directory.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants