Skip to content

chore: repoint shipped URLs from Codeberg to GitHub - #157

Closed
rubenvdlinde wants to merge 1 commit into
mainfrom
chore/retire-codeberg-urls
Closed

chore: repoint shipped URLs from Codeberg to GitHub#157
rubenvdlinde wants to merge 1 commit into
mainfrom
chore/retire-codeberg-urls

Conversation

@rubenvdlinde

Copy link
Copy Markdown
Contributor

Why

ConductionNL moved onto Codeberg on 2026-05-29 and moved back off on 2026-07-23. The git layer completed — verified 2026-08-22, zero codeberg remotes fleet-wide. The shipped metadata did not.

appinfo/info.xml is published to the Nextcloud app store, so the stale URLs are user-visible:

Field Consequence
<bugs> a bug reporter lands on an issue tracker nobody reads
<screenshot> app-store listing images 404
<repository> / <website> contributors sent to a dead mirror

What changed

Every codeberg.org/Conduction/… URL in appinfo/info.xml repointed at this repo's real GitHub URL.

The repo name was resolved from the API, not guessed from the directory. The fleet was renamed on 2026-08-21, so several apps' directory names no longer match their repo (docudeskfilinq, procestdossiq, decideskdecidiq, …).

Asset URLs were probed, not pattern-substituted. Codeberg's raw/branch/main/… does not imply GitHub's main: this fleet keeps img/ on development in several repos, so a blind main substitution would have recreated exactly the 404 this change exists to fix. Each rewritten asset URL was fetched until one returned 200. Nothing was left unresolved across the sweep.

How this was produced

Entirely through the GitHub API — no local checkout was touched. The shared apps-extra workspace has parallel sessions live in it (7 repos with uncommitted work, one with 166 unpushed commits, one detached), and reading each file from its default branch rather than from a local tree also avoided reverting fixes that had already landed remotely. 9 of the 22 repos swept turned out to be already clean on their default branch — a local-file sweep would have silently rolled those back.

Keeping it fixed

ConductionNL/.github#546 adds gate 94 retired-git-host-metadata — full-tree (deliberately not diff-scoped, since this drift is already in the tree and no PR would otherwise touch info.xml). Once merged it reaches every app with no commit in any app, via the shared quality.yml workflow.

@github-actions

Copy link
Copy Markdown
Contributor

Quality Report — ConductionNL/nextcloud-app-template @ b576e32

Check PHP Vue Security License Tests
lint
phpcs
phpmd
psalm
phpstan
phpmetrics
eslint
stylelint
build
composer ✅ 100/100
npm
app:check-code ⏭️
info.xml
REUSE
PHPUnit
Newman
Playwright ⏭️
Hydra gates ⏭️

Quality workflow — 2026-08-21 23:33 UTC

Download the full PDF report from the workflow artifacts.

@rubenvdlinde

Copy link
Copy Markdown
Contributor Author

Vervangen door een PR tegen development — deze richtte op main, wat de branch-policy (check-branch) blokkeert.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant