CUB-CI: Consolidate workflows — remove build-dev.yaml, fix dev.yml for ubuntu-latest #45

Otto · 2026-05-20T12:24:07-04:00

Otto commented

2026-05-20 12:24:07 -04:00

Problem

build-dev.yaml used actions/setup-go@v5 / actions/setup-node@v4 which are incompatible with Gitea Act runner (no node20 runtime) → always fails
Two competing workflows on same triggers → duplicate CI runs, confusing status
dev.yml used custom go-react runner label → inconsistent availability on PR merge-commits

Changes

Deleted build-dev.yaml — redundant; dev.yml already handles build + test
Fixed dev.yml:- Change runs-onfromgo-react→ubuntu-latest`
- Replace implicit Go/Node pre-install with explicit download + install (avoids incompatible setup-* actions)
- Keep docker-build-push for push events only

Evidence

Go 1.23 and Node 22 are downloaded and installed via direct tarball download (same pattern from commit ee6ad10)
npm run lint runs as part of build pipeline (was missing before the previous broke build-dev.yaml approach)
Docker push step uses secrets.ACCESS_TOKEN (already configured) for registry auth

PRs that will unblock

#44 (CUB-207) — event sync tests — was failing only on build-dev.yaml's setup-node
#42 (CUB-200) — WS gateway client — was failing on go-react runner mismatch
#39 (CUB-125) — SSE in React — was failing on go-react runner + has separate Grimm review

## Problem - `build-dev.yaml` used `actions/setup-go@v5` / `actions/setup-node@v4` which are incompatible with Gitea Act runner (no node20 runtime) → always fails - Two competing workflows on same triggers → duplicate CI runs, confusing status - `dev.yml` used custom `go-react` runner label → inconsistent availability on PR merge-commits ## Changes 1. **Deleted `build-dev.yaml`** — redundant; `dev.yml` already handles build + test 2. **Fixed `dev.yml`**:`- Change `runs-on` from `go-react`→`ubuntu-latest` - Replace implicit Go/Node pre-install with explicit download + install (avoids incompatible `setup-*` actions) - Keep docker-build-push for `push` events only ## Evidence - Go 1.23 and Node 22 are downloaded and installed via direct tarball download (same pattern from commit `ee6ad10`) - `npm run lint` runs as part of build pipeline (was missing before the previous broke `build-dev.yaml` approach) - Docker push step uses `secrets.ACCESS_TOKEN` (already configured) for registry auth ## PRs that will unblock - #44 (CUB-207) — event sync tests — was failing only on build-dev.yaml's setup-node - #42 (CUB-200) — WS gateway client — was failing on go-react runner mismatch - #39 (CUB-125) — SSE in React — was failing on go-react runner + has separate Grimm review

Otto added 3 commits 2026-05-20 12:24:08 -04:00

CUB-CI: Remove redundant build-dev.yaml — dev.yml already handles this 9e0366e780

build-dev.yaml uses actions/setup-go@v5 and actions/setup-node@v4 which
are incompatible with Gitea Act runner (no node20 runtime). dev.yml is
the canonical build workflow; having two competing workflows on the same
triggers was causing duplicate CI runs and misleading failures.

CUB-CI: Remove redundant build-dev.yaml d294818581

CUB-CI: Consolidate CI — switch to ubuntu-latest with manual Go/Node install

Dev Build & Deploy / docker-build-push (pull_request) Has been skipped

Details

Dev Build & Deploy / test-and-build (pull_request) Failing after 0s

Details

a0eb393c6c

- Remove custom go-react runner label (was inconsistent on PR branches)
- Replace with ubuntu-latest + manual Go 1.23 / Node 22 install
  (actions/setup-go and setup-node don't work with Gitea Act runner)
- Remove duplicate build-dev.yaml workflow (already deleted)
- All steps: Go test → Go build → npm ci → npm lint → npm build
- Docker push on push events only (unchanged)

Otto merged commit 48a8598d3b into dev

2026-05-20 12:24:33 -04:00

Dex referenced this issue from a commit

2026-05-20 12:30:02 -04:00

CUB-200: sync CI workflows with dev branch

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: CubeCraft-Creations/Control-Center#45