poimen-workflows

rock/poimen-workflows

Fork 0

Author	SHA1	Message	Date
Test	b958da984c	fix: standardize poimen-workflows CI to unified pattern CI / Test (pull_request) Successful in 2m7s Details CI / Build & Push Image (pull_request) Skipped Details Unified pattern enforced: - test job: runs on all branches + PRs - build-push job: only on main push, depends on test - Proper env vars (GOPRIVATE, REGISTRY, IMAGE) - Install Node.js before checkout - Install docker only in build-push - Docker login + build + push + prune	2026-09-07 00:12:35 -07:00
rockandTest	45b7f8ca61	fix: use env vars for docker registry credentials (#4 ) CI / Test (push) Successful in 2m7s Details CI / Build & Push Image (push) Failing after 1m5s Details Fix registry login by passing FORGEJO_REGISTRY_USER and FORGEJO_REGISTRY_TOKEN via environment variables instead of direct secret interpolation. Uses the proven pattern from riotpiao.com reference commit. This prevents credentials from being exposed in logs or shell history while keeping the standard docker login approach. After merge + org-level secrets configured: - All repos inherit FORGEJO_REGISTRY_USER and FORGEJO_REGISTRY_TOKEN - CI validates credentials exist before docker login - Image pushed to registry on main push --------- Co-authored-by: Test <[email protected]> Reviewed-on: #4	2026-09-07 06:48:25 +00:00
rockandTest	0261ad141b	fix: separate test and build-push jobs (#3 ) CI / Test (push) Successful in 2m24s Details CI / Build & Push Image (push) Failing after 1m7s Details ## Problem Monolithic test-build-push job runs all steps sequentially, with conditionals for push only on main. This makes it hard to see what failed and doesn't clearly separate concerns. ## Fix Split into two jobs: - test: Runs on all branches + PRs (go mod, vet, test, build binary) - build-push: Runs only on main push after test passes Move env vars to workflow level (cleaner, reused by both jobs). ## Result - PRs: test job runs ✅ (no docker install, no registry push) ✅ - Main push: test → build-push → registry push ✅ --------- Co-authored-by: Test <[email protected]> Reviewed-on: #3	2026-09-07 06:23:59 +00:00

Author

SHA1

Message

Date

Test

b958da984c

fix: standardize poimen-workflows CI to unified pattern

CI / Test (pull_request) Successful in 2m7s

Details

CI / Build & Push Image (pull_request) Skipped

Details

Unified pattern enforced:
- test job: runs on all branches + PRs
- build-push job: only on main push, depends on test
- Proper env vars (GOPRIVATE, REGISTRY, IMAGE)
- Install Node.js before checkout
- Install docker only in build-push
- Docker login + build + push + prune

2026-09-07 00:12:35 -07:00

rockandTest

45b7f8ca61

fix: use env vars for docker registry credentials (#4 )

CI / Test (push) Successful in 2m7s

Details

CI / Build & Push Image (push) Failing after 1m5s

Details

Fix registry login by passing FORGEJO_REGISTRY_USER and FORGEJO_REGISTRY_TOKEN via environment variables instead of direct secret interpolation.

Uses the proven pattern from riotpiao.com reference commit.

This prevents credentials from being exposed in logs or shell history while keeping the standard docker login approach.

After merge + org-level secrets configured:
- All repos inherit FORGEJO_REGISTRY_USER and FORGEJO_REGISTRY_TOKEN
- CI validates credentials exist before docker login
- Image pushed to registry on main push

---------

Co-authored-by: Test <[email protected]>
Reviewed-on: #4

2026-09-07 06:48:25 +00:00

rockandTest

0261ad141b

fix: separate test and build-push jobs (#3 )

CI / Test (push) Successful in 2m24s

Details

CI / Build & Push Image (push) Failing after 1m7s

Details

## Problem

Monolithic test-build-push job runs all steps sequentially, with conditionals for push only on main. This makes it hard to see what failed and doesn't clearly separate concerns.

## Fix

Split into two jobs:
- **test**: Runs on all branches + PRs (go mod, vet, test, build binary)
- **build-push**: Runs only on main push after test passes

Move env vars to workflow level (cleaner, reused by both jobs).

## Result
- PRs: test job runs ✅ (no docker install, no registry push) ✅
- Main push: test → build-push → registry push ✅

---------

Co-authored-by: Test <[email protected]>
Reviewed-on: #3

2026-09-07 06:23:59 +00:00

Compare commits

Diff Content Not Available