chore: repoint shipped URLs from Codeberg to GitHub - #159
Merged
Conversation
rubenvdlinde
requested review from
WilcoLouwerse,
bbrands02 and
rjzondervan
as code owners
August 22, 2026 06:09
Contributor
Quality Report — ConductionNL/nextcloud-app-template @
|
| Check | PHP | Vue | Security | License | Tests |
|---|---|---|---|---|---|
| lint | ✅ | ||||
| phpcs | ✅ | ||||
| phpmd | ✅ | ||||
| psalm | ✅ | ||||
| phpstan | ✅ | ||||
| phpmetrics | ✅ | ||||
| eslint | ✅ | ||||
| stylelint | ✅ | ||||
| build | ✅ | ||||
| check-specs | ✅ | ||||
| check-manifest | ✅ | ||||
| format | ✅ | ||||
| composer | ✅ | ✅ 103/103 | |||
| npm | ✅ | ✅ 720/720 | |||
| app:check-code | ⏭️ | ||||
| info.xml | ✅ | ||||
| REUSE | ❌ | ||||
| PHPUnit | ✅ | ||||
| Newman | ✅ | ||||
| Playwright | ✅ | ||||
| Hydra gates | ✅ |
Quality workflow — 2026-08-22 06:33 UTC
Download the full PDF report from the workflow artifacts.
rubenvdlinde
pushed a commit
that referenced
this pull request
Aug 22, 2026
development gained the Codeberg->GitHub repoint in #159 while this branch was open, and this branch carried the same fix — so appinfo/info.xml conflicted. Resolved in development's favour: #159 is already merged and is the version the rest of the fleet will diff against. Keeping my copy would have been a same-content fork of a file somebody else already owns. The lock bump this branch exists for (hydra-gates v1.8.2 -> v1.9.0, nc-vue 2.10.1 -> 2.11.1) is untouched. gate-94 re-run after the resolution: 0 failures.
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
1 participant
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.
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.xmlis published to the Nextcloud app store, so the stale URLs are user-visible:<bugs><screenshot><repository>/<website>What changed
Every
codeberg.org/Conduction/…URL inappinfo/info.xmlrepointed 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 (
docudesk→filinq,procest→dossiq,decidesk→decidiq, …).Asset URLs were probed, not pattern-substituted. Codeberg's
raw/branch/main/…does not imply GitHub'smain: this fleet keepsimg/ondevelopmentin several repos, so a blindmainsubstitution 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-extraworkspace 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 touchinfo.xml). Once merged it reaches every app with no commit in any app, via the sharedquality.ymlworkflow.