chore(ci): remove dead Forgejo/Codeberg CI residue - #211
Merged
Conversation
Codeberg is retired; ConductionNL is GitHub-only. GitHub Actions never executes .forgejo/**, so these workflows contributed zero status checks. Deleted 3 of 5: pre-merge-check-strict.yaml (covered by .github/workflows/code-quality.yml) and app-tests-live.yml + tests-live.yml. KEPT app-tests.yml + tests.yml -- tests.yml carries the only definition of the l10n extraction gate and the coverage ratchets (no .github/workflows/ l10n.yml here, enable-coverage-guard unset). Also repointed the .forgejo reference in tests/e2e/visual/README.md.
Contributor
Quality Report — ConductionNL/nldesign @
|
| Check | PHP | Vue | Security | License | Tests |
|---|---|---|---|---|---|
| lint | ✅ | ||||
| phpcs | ✅ | ||||
| phpmd | ✅ | ||||
| psalm | ✅ | ||||
| phpstan | ✅ | ||||
| phpmetrics | ✅ | ||||
| eslint | ✅ | ||||
| stylelint | ✅ | ||||
| build | ❌ | ||||
| composer | ✅ | ✅ 100/100 | |||
| npm | ✅ | ✅ 2/2 | |||
| PHPUnit | ✅ | ||||
| Newman | ⏭️ | ||||
| Playwright | ⏭️ |
Quality workflow — 2026-08-04 09:24 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
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
Codeberg is retired — ConductionNL is a GitHub-only organisation. GitHub Actions never executes
.forgejo/**, so every workflow under it contributed zero status checks. This removes the dead ones.Deleted (3 of 5)
.forgejo/workflows/…pre-merge-check-strict.yaml.github/workflows/code-quality.yml→ sharedConductionNL/.githubquality pipelineapp-tests-live.yml+tests-live.ymlenable-playwright/enable-newmaninputs are its GitHub homeKept on purpose — no GitHub counterpart
app-tests.yml+tests.yml—tests.ymlcarries thel10n-checkhard gate (node tests/l10n/check-l10n.js, the i18n-extraction-drift guard) plusphpunit-coverage-ratchet/frontend-coverage-ratchet. This repo has no.github/workflows/l10n.yml, does not routetest:l10nthrough the shared workflow'sfrontend-checksinput, and does not setenable-coverage-guard— so these files are the only definition of those gates. Deleting the only copy of a real gate would be a regression rather than a cleanup.Honest caveat: they do not run anywhere today — the forge that executed them is gone, and
npm run test:l10ncurrently fails ondevelopment. Porting the gate to.github/workflows/would redden CI on arrival, so it must be a deliberate follow-up with the strings backfilled, not a side effect of this cleanup.Known gap this removes (reported, not silently dropped)
app-tests-live.ymlrequestedrun-e2e/run-newman/run-visual;code-quality.ymlhere enables neither Playwright nor Newman, so that layer is currently un-gated on GitHub. Already true before this PR — the.forgejocopy has never run there.Also fixed
tests/e2e/visual/README.md— the visual project's only CI wiring was the deletedrun-visualinput; the doc now says plainly it is not wired into CI.Checks
Note for reviewers:
code-quality.ymlhas nopaths:filter, so the full shared quality suite does run on this PR even though the diff is.forgejo/**+ markdown. Any failure here is therefore pre-existingdevelopmentdebt, not caused by this diff — this change removes files GitHub Actions never read.