fix: repoint stale Codeberg links to GitHub - #333
Merged
Conversation
GitHub is the only host for this org. Codeberg was a mirror and is no longer used, including for issue tracking, so these URLs were dead. Changes (both target `nextcloud-vue`, whose slug did NOT change in the fleet rename — only the host and org were wrong): - docs/features/widgets.md: 3 links to the shared `@conduction/nextcloud-vue` repo -> github.com/ConductionNL/nextcloud-vue - src/manifest.json: `$schema` `/raw/branch/main/` -> `/raw/main/` Deliberately NOT rewritten — the 21 archived Hydra run records (openspec/changes/archive/*/hydra.json), each carrying `"repo": "https://codeberg.org/Conduction/launchpad"`: These are immutable historical pipeline records, and each pairs that `repo` field with an `"issue"` NUMBER from the Codeberg era (5-39 across 20 of the 21 records). Codeberg issue numbers do NOT correspond to GitHub issue numbers. Repointing `repo` at GitHub while leaving those numbers in place would make every record silently assert a REAL BUT UNRELATED GitHub issue -- the same hazard as rewriting an issue URL's host, just split across two fields. Left intact as Codeberg-era records. Also left for a human decision: - .forgejo/workflows/*.yml (3 files) -- Forgejo CI config, including a `CODEBERG_TOKEN` secret reference in release-stable.yml - docs/migration/widget-library-to-ncvue.md -- prose describing the historical "publish a beta via Codeberg CI" step, not a link
rubenvdlinde
requested review from
Rem-Dam,
SudoThijn and
remko48
as code owners
August 24, 2026 08:42
Contributor
Quality Report — ConductionNL/launchpad @
|
| Check | PHP | Vue | Security | License | Tests |
|---|---|---|---|---|---|
| lint | ✅ | ||||
| phpcs | ✅ | ||||
| phpmd | ✅ | ||||
| psalm | ✅ | ||||
| phpstan | ✅ | ||||
| phpmetrics | ✅ | ||||
| eslint | ✅ | ||||
| stylelint | ✅ | ||||
| build | ✅ | ||||
| check-manifest | ✅ | ||||
| format | ✅ | ||||
| check-schema-l10n | ✅ | ||||
| composer | ✅ | ✅ 104/104 | |||
| npm | ✅ | ✅ 538/538 | |||
| app:check-code | ⏭️ | ||||
| info.xml | ✅ | ||||
| REUSE | ❌ | ||||
| PHPUnit | ✅ | ||||
| Newman | ✅ | ||||
| Playwright | ✅ | ||||
| Hydra gates | ✅ |
Quality workflow — 2026-08-24 10:03 UTC
Download the full PDF report from the workflow artifacts.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
GitHub is the only host for this org. Codeberg was a mirror and is not used, including for issues — these URLs were dead.
Both edits target
nextcloud-vue, whose slug did not change in the fleet rename; only the host and org (Conduction→ConductionNL) were wrong.Counts by category
docs/features/widgets.md→github.com/ConductionNL/nextcloud-vueraw/branch/→raw/)src/manifest.json$schema2 files changed, 4 insertions, 4 deletions.
The issue-number decision — why 21 files were NOT changed
A naive sweep would have rewritten 21 archived Hydra run records (
openspec/changes/archive/*/hydra.json), each carrying:These are immutable historical pipeline records, and each pairs that
repofield with anissuenumber from the Codeberg era — 5 to 39 across 20 of the 21 records. Codeberg issue numbers do not correspond to GitHub issue numbers. Repointingrepoat GitHub while leaving those numbers in place would make every record silently assert a real but unrelated GitHub issue — the same hazard as rewriting an issue URL's host, just split across two fields instead of contained in one.They are therefore left intact as Codeberg-era records. This is the main reason this PR is 2 files rather than the ~25 a grep would suggest.
Also deliberately left
.forgejo/workflows/(3 files:l10n-parity.yml,pre-merge-check-strict.yaml,release-stable.yml) — Forgejo CI config, not deleted. Noterelease-stable.ymlreferences aCODEBERG_TOKENsecret; if Codeberg is fully retired that secret and these workflows want a human decision.docs/migration/widget-library-to-ncvue.md:112— prose describing the historical "publish a beta (Codeberg CI)" step. A record of how it was done, not a link.Verification
git grep -in codeberg→ zerocodeberg.orgURLs remain outside the archived run records. The 5 remaining non-archive hits are the deliberate keeps above.src/manifest.jsonre-parsed as JSON → valid.curl -L→ HTTP 200 (the repo page and theraw/mainschema URL).lib/files touched, sophp -l/phpcsare not applicable to this change.