Skip to content

ci: move workflow jobs to sm-standard-2 self-hosted runners - #252

Merged
overtrue merged 1 commit into
mainfrom
ci/runner-labels
Sep 30, 2026
Merged

overtrue merged 1 commit into
mainfrom
ci/runner-labels

Conversation

@majinghe

Copy link
Copy Markdown
Contributor

Pull Request

Description

Point every workflow job at the sm-standard-2 self-hosted runners instead of ubuntu-latest. CI-only change; no application code is touched.

The console has no Docker build jobs (no Dockerfile and no docker commands or actions in any workflow), so the dind runner pool is not needed here — all 19 jobs across build.yml, quality.yml, test.yml, release.yml, and dependencies.yml run on sm-standard-2.

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update
  • Code refactoring
  • Performance improvement
  • Test improvements
  • Security fix

Testing

Changed only runs-on: labels; all local quality gates pass.

  • Unit tests added/updated — N/A (no code change)
  • Manual testing completed
# YAML validation and job/runner mapping review
python3 -c "import yaml,glob;[yaml.safe_load(open(f)) for f in glob.glob('.github/workflows/*.yml')]"

# Repository quality gates (node v22.23.1)
pnpm install --frozen-lockfile
pnpm type-check
pnpm lint
pnpm format:check
pnpm test:run

All gates passed, including tests/lib/quality-workflow-source.test.js, which asserts the quality.yml formatting job contents.

Checklist

  • Code follows the project's style guidelines
  • Self-review completed
  • TypeScript types are properly defined — N/A (no code change)
  • All commit messages are in English (Conventional Commits)
  • All existing tests pass
  • No new dependencies added, or they are justified

Related Issues

N/A

Point every workflow job (build, quality, test, release, dependencies)
at the sm-standard-2 self-hosted runners instead of ubuntu-latest. The
console has no Docker build jobs, so the dind runner pool is not needed.
@majinghe
majinghe requested a review from overtrue September 30, 2026 06:54
@overtrue
overtrue merged commit 95edf52 into main Sep 30, 2026
10 checks passed
@overtrue
overtrue deleted the ci/runner-labels branch September 30, 2026 07:19
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.

2 participants