merge: fix CI workflow - add Node.js and docker.io installs (#17)
CI / Test (push) Successful in 2m23s
CI / Build & Push Image (push) Failing after 49s

Merge fix/memory-ci-nodejs-docker into main to enable CI triggers.

## Changes
- Add Node.js install before actions/checkout@v4
- Add docker.io install before docker login
- Add env vars (REGISTRY, REGISTRY_USER)
- Test job runs on all branches + PRs 
- Build-push job only runs on main push 

## Result
- PRs: CI runs tests (no registry push) 
- Main push: CI runs tests + builds + pushes to registry Reviewed-on: rock/poimen-memory#17

Co-authored-by: rock <[email protected]>
This commit was merged in pull request #17.
This commit is contained in:
2026-09-07 06:24:18 +00:00
committed by rock
parent d8f8ad3347
commit 2bbcc6eef9
2 changed files with 40 additions and 26 deletions
+1
View File
@@ -99,3 +99,4 @@ See `config/default.toml` for:
6. Document in API.md
See `CLAUDE.md` for project context and constraints.
# CI test 1788759975