fix: align CI push target to westsidekingsandqueens Harbor project (#100) #101

Merged

forgejo_admin merged 1 commit from 100-fix-harbor-image-repo into main

2026-03-27 05:01:34 +00:00

forgejo_admin commented

2026-03-27 04:57:38 +00:00

Owner

Summary

CI was pushing the built image to westside-app/app in Harbor, but the deployment pulls from westsidekingsandqueens/app
This one-line fix aligns the CI push target to the correct Harbor project

Changes

.woodpecker.yaml: Changed repo: westside-app/app to repo: westsidekingsandqueens/app

Test Plan

Tests pass locally
Next CI pipeline on main pushes image to westsidekingsandqueens/app in Harbor
ArgoCD picks up the new image and deploys successfully

Review Checklist

Passed automated review-fix loop
No secrets committed
No unnecessary file changes
Commit messages are descriptive

forgejo_admin/westside-app #100 -- the Forgejo issue this PR fixes
westside-app -- the project this work belongs to

Closes #100

## Summary - CI was pushing the built image to `westside-app/app` in Harbor, but the deployment pulls from `westsidekingsandqueens/app` - This one-line fix aligns the CI push target to the correct Harbor project ## Changes - `.woodpecker.yaml`: Changed `repo: westside-app/app` to `repo: westsidekingsandqueens/app` ## Test Plan - [ ] Tests pass locally - [ ] Next CI pipeline on main pushes image to `westsidekingsandqueens/app` in Harbor - [ ] ArgoCD picks up the new image and deploys successfully ## Review Checklist - [ ] Passed automated review-fix loop - [ ] No secrets committed - [ ] No unnecessary file changes - [ ] Commit messages are descriptive ## Related Notes - `forgejo_admin/westside-app #100` -- the Forgejo issue this PR fixes - `westside-app` -- the project this work belongs to Closes #100

forgejo_admin added 1 commit

2026-03-27 04:57:38 +00:00

fix: align CI push target to westsidekingsandqueens Harbor project

ci/woodpecker/pr/woodpecker Pipeline was successful

Details

27257b342c

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

forgejo_admin commented

2026-03-27 04:58:20 +00:00

Author

Owner

QA Review -- PR #101

Scope: 1 file changed, 1 addition, 1 deletion.

Diff Analysis

The change is a single-line fix in .woodpecker.yaml:

-      repo: westside-app/app
+      repo: westsidekingsandqueens/app

Review Findings

Correctness: The Harbor project name westsidekingsandqueens aligns with the deployment target. This fixes the image push/pull mismatch described in #100.
Scope containment: No other files or lines were modified. No clone step changes, no registry URL changes, no insecure setting changes.
No secrets committed: Confirmed -- credentials remain in from_secret references.
Commit message: Descriptive and follows conventional commit format.

No Issues Found

This is a straightforward one-line config fix with zero risk of side effects.

VERDICT: APPROVE

## QA Review -- PR #101 **Scope:** 1 file changed, 1 addition, 1 deletion. ### Diff Analysis The change is a single-line fix in `.woodpecker.yaml`: ```diff - repo: westside-app/app + repo: westsidekingsandqueens/app ``` ### Review Findings - **Correctness:** The Harbor project name `westsidekingsandqueens` aligns with the deployment target. This fixes the image push/pull mismatch described in #100. - **Scope containment:** No other files or lines were modified. No clone step changes, no registry URL changes, no insecure setting changes. - **No secrets committed:** Confirmed -- credentials remain in `from_secret` references. - **Commit message:** Descriptive and follows conventional commit format. ### No Issues Found This is a straightforward one-line config fix with zero risk of side effects. --- **VERDICT: APPROVE**