Skip to content

meta: skip scheduled workflows on forks#63565

Merged
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
JamieMagee:guard-scheduled-workflows-on-forks
May 27, 2026
Merged

meta: skip scheduled workflows on forks#63565
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
JamieMagee:guard-scheduled-workflows-on-forks

Conversation

@JamieMagee
Copy link
Copy Markdown
Contributor

Scheduled workflows run on forks by default, which wastes minutes and spams fork owners with failure emails. Most workflows here already guard against that (tools.yml, stale.yml, etc.). This adds the same if: github.repository == 'nodejs/node' to the three that don't: daily.yml, codeql.yml, scorecard.yml. Manual workflow_dispatch runs still go through.

@nodejs-github-bot
Copy link
Copy Markdown
Collaborator

Review requested:

  • @nodejs/actions

@nodejs-github-bot nodejs-github-bot added the meta Issues and PRs related to the general management of the project. label May 25, 2026
Guards scheduled jobs in daily.yml, codeql.yml, and scorecard.yml so
they only run on nodejs/node, matching the pattern already used in
tools.yml, stale.yml, and others. This prevents wasted Actions minutes
and failed-run email notifications on forks.

Signed-off-by: Jamie Magee <jamie.magee@gmail.com>
@JamieMagee JamieMagee force-pushed the guard-scheduled-workflows-on-forks branch from 903ef4b to d0e2c4b Compare May 25, 2026 18:04
@aduh95 aduh95 added author ready PRs that have at least one approval, no pending requests for changes, and a CI started. commit-queue Add this label to land a pull request using GitHub Actions. labels May 25, 2026
@nodejs-github-bot nodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label May 27, 2026
@nodejs-github-bot nodejs-github-bot merged commit 47d964b into nodejs:main May 27, 2026
21 checks passed
@nodejs-github-bot
Copy link
Copy Markdown
Collaborator

Landed in 47d964b

@JamieMagee JamieMagee deleted the guard-scheduled-workflows-on-forks branch May 27, 2026 18:24
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. meta Issues and PRs related to the general management of the project.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants