poimen-memory

Author	SHA1	Message	Date
rock	4ed5c76865	fix: use env vars for docker registry credentials CI / Test (pull_request) Successful in 2m5s Details CI / Build & Push Image (pull_request) Skipped Details	2026-09-06 23:40:56 -07:00
rock	ba86c3cedb	fix: use env vars for docker registry credentials CI / Test (pull_request) Successful in 2m21s Details CI / Build & Push Image (pull_request) Skipped Details	2026-09-06 23:38:01 -07:00
rock	c3f5f65540	fix: validate registry credentials before docker login CI / Test (pull_request) Successful in 2m8s Details CI / Build & Push Image (pull_request) Skipped Details Problem: Registry login fails silently if secrets aren't configured - Empty FORGEJO_REGISTRY_USER/TOKEN → docker login hangs/fails - No clear error message about missing credentials Solution: Add validation step that checks credentials exist - Fails early with clear error if secrets missing - Shows how to configure in repo settings - Uses direct secret injection (not via env vars) - Isolates docker config to /tmp/docker-config Result: CI will fail fast with actionable error if credentials missing	2026-09-06 23:34:48 -07:00
rock	2bbcc6eef9	merge: fix CI workflow - add Node.js and docker.io installs (#17 ) CI / Test (push) Successful in 2m23s Details CI / Build & Push Image (push) Failing after 49s Details 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]>	2026-09-07 06:24:18 +00:00

Compare commits

Diff Content Not Available