Add Woodpecker CI pipeline #7

New issue

Closed

opened 2026-05-09 18:01:49 +00:00 by forgejo_admin · 3 comments

forgejo_admin commented

2026-05-09 18:01:49 +00:00

Contributor

Superseded

Closed — superseded by #19 (Woodpecker CI with test gates + Arch base image). PR #16 also closed.

### Superseded Closed — superseded by #19 (Woodpecker CI with test gates + Arch base image). PR #16 also closed.

forgejo_admin referenced this issue

2026-05-09 18:23:03 +00:00

Scaffold Rails app with landing page, contact form, and Keycloak SSO dashboard #1

forgejo_admin commented

2026-05-09 18:57:24 +00:00

Author

Contributor

Scope Review: NEEDS_REFINEMENT

Review note: review-1185-2026-05-09

Template is complete and file targets verified. Two issues found:

[BODY] Harbor URL is wrong in AC2. The AC says harbor-core.harbor.svc.cluster.local but every existing working pipeline (basketball-api, westside-admin, westside-contracts, notion-mcp-remote) uses harbor.harbor.svc.cluster.local. Following the AC literally will cause CI failure. Fix the URL in the acceptance criteria.
[SCOPE] Missing arch note. The arch:rails-app label has no backing architecture note in pal-e-docs. Create arch-rails-app note to complete traceability.

## Scope Review: NEEDS_REFINEMENT Review note: `review-1185-2026-05-09` Template is complete and file targets verified. Two issues found: - **[BODY] Harbor URL is wrong in AC2.** The AC says `harbor-core.harbor.svc.cluster.local` but every existing working pipeline (basketball-api, westside-admin, westside-contracts, notion-mcp-remote) uses `harbor.harbor.svc.cluster.local`. Following the AC literally will cause CI failure. Fix the URL in the acceptance criteria. - **[SCOPE] Missing arch note.** The `arch:rails-app` label has no backing architecture note in pal-e-docs. Create `arch-rails-app` note to complete traceability.

forgejo_admin commented

2026-05-10 02:38:02 +00:00

Author

Contributor

Scope Review: NEEDS_REFINEMENT

Review note: review-1185-2026-05-09

Template is complete and file targets are verified. One traceability gap found.

[SCOPE] Architecture note arch-rails-app does not exist in pal-e-docs. The arch:rails-app label is used across multiple board items but has no backing note. Create arch-rails-app to complete the traceability triangle.

All other checks pass: story note verified, repo placement correct, dependencies documented, acceptance criteria testable, no decomposition needed.

## Scope Review: NEEDS_REFINEMENT Review note: `review-1185-2026-05-09` Template is complete and file targets are verified. One traceability gap found. - **[SCOPE]** Architecture note `arch-rails-app` does not exist in pal-e-docs. The `arch:rails-app` label is used across multiple board items but has no backing note. Create `arch-rails-app` to complete the traceability triangle. All other checks pass: story note verified, repo placement correct, dependencies documented, acceptance criteria testable, no decomposition needed.

forgejo_admin commented

2026-05-10 02:42:21 +00:00

Author

Contributor

Scope Review: APPROVED

Review note: review-1185-2026-05-09

Re-review after refinement. The sole blocker from v1 (arch-rails-app note missing) is resolved -- note exists at slug arch-rails-app (tagged architecture,active, project pal-enterprises). All 11 template sections present, all 3 traceability legs verified, all file targets confirmed. Ticket is ready to advance.

## Scope Review: APPROVED Review note: `review-1185-2026-05-09` Re-review after refinement. The sole blocker from v1 (`arch-rails-app` note missing) is resolved -- note exists at slug `arch-rails-app` (tagged architecture,active, project pal-enterprises). All 11 template sections present, all 3 traceability legs verified, all file targets confirmed. Ticket is ready to advance.

forgejo_admin referenced this issue from a pull request that will close it,

2026-05-10 15:55:48 +00:00

Add Woodpecker CI pipeline #16

forgejo_admin referenced this issue

2026-05-10 15:56:47 +00:00

Add Woodpecker CI pipeline #16