Skip to content

Move validate/publish to ubuntu-latest; release 2.0.53 - #13

Merged
robgilbreath merged 3 commits into
masterfrom
chore/repo-standard/hosted-runners
Sep 1, 2026
Merged

robgilbreath merged 3 commits into
masterfrom
chore/repo-standard/hosted-runners

Conversation

@robgilbreath

@robgilbreath robgilbreath commented Sep 1, 2026 •

Copy link
Copy Markdown

Why

The org's self-hosted runner group has allows_public_repositories: false (correctly — it keeps fork PRs off the fleet), so this public repo's runs-on: self-hosted jobs can never be scheduled. Every Validate run since 2026-06-19 has been cancelled/queued; the two open Dependabot PRs (#10, #11) are among them.

What changed

  • validate.yml and npm-publish.yml: runs-on: self-hosted -> runs-on: ubuntu-latest.
  • validate.yml: add actions/setup-node (node-version-file: .nvmrc) before verify-node-toolchain. That vendored action asserts node/npm majors against the org vars (NODE_MAJOR_VERSION=24 / NPM_MAJOR_VERSION=11) and previously relied on the self-hosted box having Node 24 preinstalled; on the hosted image it would have checked Ubuntu's system Node. Node 24 bundles npm 11, so the check passes unchanged. validate-codebase runs setup-node again afterwards — cache hit, harmless.
  • Comments updated: the same-repo if gate is now a policy choice rather than a runner-safety requirement, matching datto-rmm-api-client.
  • package.json 2.0.52 -> 2.0.53: the validate gate requires a version bump on every PR (no Dependabot exemption, per Apply version-bump gate to Dependabot PRs too (release 2.0.52) #9), so this ships as a CI-only release, same as Apply version-bump gate to Dependabot PRs too (release 2.0.52) #9 did.

Nothing in either workflow depended on runner-local state: publish auth is OIDC trusted publishing (id-token: write), npm ci hits registry.npmjs.org, and no cached paths or local registries are referenced. datto-rmm-api-client runs the same steps on ubuntu-latest successfully.

Refs pncit/.github#3

🤖 Generated with Claude Code

Also pins npm@12.0.2 in publish (was floating on @latest).

robgilbreath and others added 2 commits September 1, 2026 11:36
…heck

The org runner group rejects jobs from public repos, so `runs-on:
self-hosted` has left every run since 2026-06-19 unschedulable. Switch
both workflows to GitHub-hosted runners and install the .nvmrc Node
ahead of verify-node-toolchain so it checks our toolchain, not the
image's system Node.

Refs pncit/.github#3

Co-Authored-By: Claude Code <noreply@anthropic.com>
CI-only release: every PR into master must bump the version (validate.yml
gate), and this one moves the workflows to GitHub-hosted runners.

Refs pncit/.github#3

Co-Authored-By: Claude Code <noreply@anthropic.com>
Was floating on @latest. Org standard is npm 12 (matches shared-actions
validate-codebase, which pins 12.0.2); npm/cli#9722 was fixed in 12.0.1.

Co-Authored-By: Claude Code <noreply@anthropic.com>
@robgilbreath
robgilbreath merged commit 46342de into master Sep 1, 2026
1 check passed
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