chore: remove per-repo clone URL override #40

Merged

forgejo_admin merged 1 commit from 39-chore-remove-per-repo-clone-url-override into main

2026-03-14 22:32:59 +00:00

forgejo_admin commented

2026-03-14 22:25:17 +00:00

Owner

Summary

Remove the per-repo clone: section from .woodpecker.yml. The global WOODPECKER_FORGEJO_CLONE_URL (set in pal-e-platform PR #56) now handles clone routing centrally, making per-repo overrides redundant.

Changes

.woodpecker.yml — removed the clone: block (5 lines + blank line separator). All other pipeline config (lint, publish steps) unchanged. The publish step's FORGEJO_PYPI_URL referencing forgejo-http.forgejo.svc.cluster.local is intentionally kept — that is the package registry URL, not a clone URL.

Test Plan

Woodpecker pipeline triggers on this PR and clone step succeeds using the global WOODPECKER_FORGEJO_CLONE_URL
Lint step passes
No unrelated changes

Review Checklist

Only the clone: block was removed
publish step FORGEJO_PYPI_URL is untouched
Blank line separation between remaining top-level keys preserved
CI pipeline passes

Plan: todo-remove-clone-url-overrides (no plan ancestry)
Forgejo issue: #39
pal-e-platform PR #56 — set global clone URL

Closes #39

## Summary Remove the per-repo `clone:` section from `.woodpecker.yml`. The global `WOODPECKER_FORGEJO_CLONE_URL` (set in pal-e-platform PR #56) now handles clone routing centrally, making per-repo overrides redundant. ## Changes - `.woodpecker.yml` — removed the `clone:` block (5 lines + blank line separator). All other pipeline config (lint, publish steps) unchanged. The `publish` step's `FORGEJO_PYPI_URL` referencing `forgejo-http.forgejo.svc.cluster.local` is intentionally kept — that is the package registry URL, not a clone URL. ## Test Plan - [ ] Woodpecker pipeline triggers on this PR and clone step succeeds using the global `WOODPECKER_FORGEJO_CLONE_URL` - [ ] Lint step passes - [ ] No unrelated changes ## Review Checklist - [x] Only the `clone:` block was removed - [x] `publish` step `FORGEJO_PYPI_URL` is untouched - [x] Blank line separation between remaining top-level keys preserved - [ ] CI pipeline passes ## Related - Plan: `todo-remove-clone-url-overrides` (no plan ancestry) - Forgejo issue: #39 - pal-e-platform PR #56 — set global clone URL Closes #39

forgejo_admin added 1 commit

2026-03-14 22:25:17 +00:00

chore: remove per-repo clone URL override from .woodpecker.yml

ci/woodpecker/push/woodpecker Pipeline was successful

Details

ci/woodpecker/pr/woodpecker Pipeline was successful

Details

082fd08f68

The global WOODPECKER_FORGEJO_CLONE_URL (set in pal-e-platform PR #56)
now handles clone routing. The per-repo clone section was originally
added in PR #15 to work around a TLS issue that has since been resolved.

Closes #39

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

forgejo_admin commented

2026-03-14 22:25:34 +00:00

Author

Owner

Review: LGTM

Diff: 1 file changed, 0 additions, 6 deletions.

Findings: None. The change is exactly scoped:

Only the clone: block (5 lines + trailing blank line) was removed
The when: and steps: sections are untouched
The publish step's FORGEJO_PYPI_URL referencing forgejo-http.forgejo.svc.cluster.local is correctly preserved (package registry, not clone URL)
Blank line separation between when: and steps: is maintained
PR body has Closes #39 for auto-close
No unrelated changes

Verdict: Ship it. Pending CI pipeline confirmation.

## Review: LGTM **Diff:** 1 file changed, 0 additions, 6 deletions. **Findings:** None. The change is exactly scoped: - Only the `clone:` block (5 lines + trailing blank line) was removed - The `when:` and `steps:` sections are untouched - The `publish` step's `FORGEJO_PYPI_URL` referencing `forgejo-http.forgejo.svc.cluster.local` is correctly preserved (package registry, not clone URL) - Blank line separation between `when:` and `steps:` is maintained - PR body has `Closes #39` for auto-close - No unrelated changes **Verdict:** Ship it. Pending CI pipeline confirmation.