Skip to content

ci: switch GitHub-hosted Ubuntu runners to depot-ubuntu-24.04 - #93

Open
claude[bot] wants to merge 1 commit into
mainfrom
claude/depot-runners
Open

ci: switch GitHub-hosted Ubuntu runners to depot-ubuntu-24.04#93
claude[bot] wants to merge 1 commit into
mainfrom
claude/depot-runners

Conversation

@claude

@claude claude Bot commented Sep 14, 2026

Copy link
Copy Markdown
Contributor

Requested by Matanya Loewenthal · Slack thread

GitHub-hosted CI runners are down for the org, so this repo's CI can't run. This moves every GitHub-hosted Ubuntu runner to Depot's depot-ubuntu-24.04 so CI runs again. Worth reviewing promptly — CI is blocked until it merges.

Before: All six jobs across five workflows ran on runs-on: ubuntu-latest, GitHub-hosted runners that are currently down.

After: All six jobs run on runs-on: depot-ubuntu-24.04. No other workflow logic changed; every touched file still parses as valid YAML.

Workflows/jobs touched:

  • checks.ymltest (Node 20/22/24 matrix, matrix itself unchanged)
  • release.ymlrelease, homebrew
  • cut-release.ymlcut
  • install-smoke.ymlsmoke
  • auto-approve-claude-lgtm.ymlapprove

No macOS or Windows runners in this repo.

🤖 Generated with Claude Code

https://claude.ai/code/session_01UbHKWrDays8Yjsw5JQUYpo


Generated by Claude Code

@claude

claude Bot commented Sep 14, 2026

Copy link
Copy Markdown
Contributor Author

The failing check checks / Test & typecheck (24.x) is pre-existing spec drift, not this PR's. The live spec at api.polylane.com/v1/doc no longer contains the skills routes, so the codegen step (pretypecheck) drops the five skills* client methods and src/commands/skill/* fails TS2339 on every branch — reproduced locally on a fresh install with identical errors. On the Depot runner, checkout, Node setup, install, and codegen all succeeded, so the runner switch itself works.

The fix is a separate PR removing src/commands/skill/, mirroring how #91 handled the scan_reports removal; it's awaiting the go-ahead.


Generated by Claude Code

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