Skip to content

ci: default secret-scan, dependabot-auto-merge and lint-workflows to self-hosted - #89

Open
nkg wants to merge 1 commit into
mainfrom
ci/self-hosted-defaults-remaining
Open

nkg wants to merge 1 commit into
mainfrom
ci/self-hosted-defaults-remaining

Conversation

@nkg

@nkg nkg commented Sep 20, 2026

Copy link
Copy Markdown
Owner

Companion to #88. That one covers claude-code-review.yml; this covers everything else that still relies on a hosted default.

The audit

Every uses: nkg/github-actions/.github/workflows/* call across HordiaLabs, Sproncy and Regularmusic — 201 calls — checked for whether the calling job passes runs-on:

Reusable Calls landing on GitHub-hosted runners
claude-code-review.yml 43 → #88
secret-scan.yml 6
dependabot-auto-merge.yml 4
lint-workflows.yml 1

54 calls across 43 repos. With credits exhausted, none of them start — each fails ~2s after queueing with "The job was not started because recent account payments have failed or your spending limit needs to be increased".

The 11 non-review ones are all in Sproncy:

  • secret-scan.yml — GitHub-runners, agent-skills, sproncy-schemas, sproncy-secrets-dashboard, sproncy-transcription, sproncy-website-scrapers
  • dependabot-auto-merge.yml — sproncy-schemas, sproncy-secrets-dashboard, sproncy-transcription, sproncy-website-scrapers
  • lint-workflows.yml — agent-skills

Why flip the default rather than patch six repos

Every other consumer of these three already passes runs-on explicitly, so the default only reaches stubs that omitted it — and all 11 pin @v3, so they pick this up the moment release.yml moves the floating major tag. One change, no per-repo edits, no Dependabot wait. (Contrast #88, where 10 of the 43 are SHA-pinned and will lag until their pins bump.)

The other 24 reusables keep ["ubuntu-latest"] — nothing currently depends on their default, so flipping them would be churn without a consumer.

Public repos

Descriptions now read "Defaults to the self-hosted pool. Public/open-source repos opt out by passing ["ubuntu-latest"]", matching claude.yml. The opt-out matters for more than free minutes: fork PRs on a self-hosted pool are the hazard GitHub warns about. No public repo consumes any of these three today — checked the three orgs and nkg/*.

🤖 Generated with Claude Code

https://claude.ai/code/session_015wGZqkEacSGCx8watYCm1b

…self-hosted

Companion to the claude-code-review default flip. An audit of every
`uses: nkg/github-actions/.github/workflows/*` call across HordiaLabs,
Sproncy and Regularmusic -- 201 calls in total -- found 54 that take a
hosted default, and with Actions credits exhausted those jobs no longer
start at all.

43 of the 54 are claude-code-review. The other 11 are these three
reusables, all in Sproncy:

  secret-scan.yml            6  GitHub-runners, agent-skills, sproncy-schemas,
                                sproncy-secrets-dashboard, sproncy-transcription,
                                sproncy-website-scrapers
  dependabot-auto-merge.yml  4  sproncy-schemas, sproncy-secrets-dashboard,
                                sproncy-transcription, sproncy-website-scrapers
  lint-workflows.yml         1  agent-skills

Every other consumer of these three already passes `runs-on` explicitly, so
this changes behaviour only for the stubs that relied on the default -- and
all 11 pin `@v3`, so they pick the change up as soon as release.yml moves the
floating major tag. No per-repo edits needed.

The other 24 reusables keep their `["ubuntu-latest"]` default: nothing
currently depends on it, so flipping them would be churn without a consumer.

Public repos should pass '["ubuntu-latest"]' -- fork PRs on a self-hosted
pool are the hazard GitHub warns about, and the descriptions now say so. No
public repo consumes any of these today; checked the three orgs plus nkg/*.
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.

1 participant