[ISSUE-75] Update workflow trigger to push on test/main branches #3

Dave · 2026-07-04T19:03:16Z

Dave commented

2026-07-04 19:03:16 +00:00

Summary

Change the CI runner test workflow trigger from workflow_dispatch only to also include push on test and main branches, enabling automatic execution when code is pushed.

Changes

.gitea/workflows/test-ci-runner.yml: Added push trigger targeting test and main branches while retaining workflow_dispatch for manual triggering

Issues

Hithomelabs/HomeLabDocker#75

Testing

Push to test branch triggers workflow automatically
Push to main branch triggers workflow automatically
Manual workflow_dispatch still works via UI

## Summary Change the CI runner test workflow trigger from `workflow_dispatch` only to also include `push` on `test` and `main` branches, enabling automatic execution when code is pushed. ## Changes - `.gitea/workflows/test-ci-runner.yml`: Added `push` trigger targeting `test` and `main` branches while retaining `workflow_dispatch` for manual triggering ## Issues - Hithomelabs/HomeLabDocker#75 ## Testing - [ ] Push to `test` branch triggers workflow automatically - [ ] Push to `main` branch triggers workflow automatically - [ ] Manual `workflow_dispatch` still works via UI

Dave added 1 commit 2026-07-04 19:03:16 +00:00

Hithomelabs/ci-runner-test#75 : Update workflow trigger to push on test/main branches d97080e484

Dave referenced this pull request

2026-07-04 19:03:23 +00:00

[ISSUE-75] Sync main -> test: Update workflow trigger #4

Linus approved these changes 2026-07-04 19:05:16 +00:00

Linus left a comment

PR Review: [ISSUE-75] Update workflow trigger to push on test/main branches

Code Quality

Code follows project conventions
No obvious bugs or anti-patterns
Error handling is appropriate (N/A - config change)

Security

No exposed secrets
Input validation present (N/A)
Appropriate access controls

Testing

Change is configuration-only; runtime testing covered in AC

Review Verdict

The YAML change matches the architect's spec from Issue #75 exactly:

on:
  push:
    branches: [test, main]
  workflow_dispatch:

✅ push trigger added for test and main branches as specified
✅ workflow_dispatch retained (per AC: "manual triggering remains available as a fallback")
✅ Clean, minimal change with no unintended modifications

Approval Status

APPROVED — Ready for merge (squash recommended)

## PR Review: [ISSUE-75] Update workflow trigger to push on test/main branches ### Code Quality - [x] Code follows project conventions - [x] No obvious bugs or anti-patterns - [x] Error handling is appropriate (N/A - config change) ### Security - [x] No exposed secrets - [x] Input validation present (N/A) - [x] Appropriate access controls ### Testing - [x] Change is configuration-only; runtime testing covered in AC ### Review Verdict The YAML change matches the architect's spec from Issue #75 exactly: ```yaml on: push: branches: [test, main] workflow_dispatch: ``` - ✅ `push` trigger added for `test` and `main` branches as specified - ✅ `workflow_dispatch` retained (per AC: "manual triggering remains available as a fallback") - ✅ Clean, minimal change with no unintended modifications ### Approval Status **APPROVED** — Ready for merge (squash recommended)

hitanshu requested review from hitanshu 2026-07-04 19:06:17 +00:00

hitanshu approved these changes 2026-07-04 19:06:32 +00:00

hitanshu merged commit 27c73459c0 into main

2026-07-04 19:06:39 +00:00

hitanshu referenced this issue from a commit

2026-07-04 19:06:40 +00:00

[ISSUE-75] Update workflow trigger to push on test/main branches (#3)

Sign in to join this conversation.

No reviewers