feat(pages): het handboek publiceert weer, nu op GitHub Pages - #5
Open
MWest2020 wants to merge 1 commit into
Open
feat(pages): het handboek publiceert weer, nu op GitHub Pages#5MWest2020 wants to merge 1 commit into
MWest2020 wants to merge 1 commit into
Conversation
Sinds 2026-08-03 had de pipeline geen deploy-stap, met een goede reden: publiceren naar Codeberg zou een Codeberg-PAT als GitHub-secret vergen. Die reden vervalt nu de site zelf naar GitHub verhuist — Pages werkt met het ingebouwde token en OIDC, dus er komt geen nieuw secret bij. site_url om, artifact-upload aan het eind van build, en een aparte deploy-job met environment github-pages. needs: build betekent dat een rode gate niets live zet; alleen een push naar main publiceert. De linkcheck meldt tot de eerste publicatie ~181 verwijzingen naar conductionnl.github.io die nog niet bestaan. Hij is niet blokkerend, en dat aantal terug naar nul is precies de voorwaarde om hem hard te zetten. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
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.
Sinds 2026-08-03 had de pipeline geen deploy-stap, met een goede reden: publiceren naar Codeberg zou een Codeberg-PAT als GitHub-secret vergen — een credential over twee forges heen. Die reden vervalt nu de site zelf naar GitHub verhuist: Pages werkt met het ingebouwde token en OIDC, dus er komt geen enkel nieuw secret bij.
mkdocs.yml:site_urlnaarhttps://conductionnl.github.io/handbook/. Dat veld stuurt de canonical-links ensitemap.xml— die wezen naar de bevroren Codeberg-site en waren precies de 6 bevindingen die de gerepareerde linkcheck meldde.docs.yml:pages: write+id-token: write, eenconcurrency: pages-groep die lopende deploys niet afbreekt, een artifact-upload aan het eind vanbuild, en een apartedeploy-job metenvironment: github-pages. Actions gepind op commit-SHA, zoals de rest van dit bestand.main, enneeds: buildbetekent dat een rode gate niets live zet. Een PR bouwt en gate't wel.docs/org/pipeline-rood.md: de sectie "De site is bevroren" vervangen door waar de site staat en wat wel en niet publiceert.Wat je in de run gaat zien. De linkcheck meldt tot de eerste publicatie ~181 verwijzingen naar
conductionnl.github.iodie nog niet bestaan; de 6 Codeberg-404's zijn weg. Hij is niet blokkerend. Dat aantal terug naar nul ná de eerste deploy is precies de voorwaarde om hem hard te zetten — dat is een aparte wijziging.Lokaal: 27 tests,
mkdocs build --strict, freshness 0/90, sitemap wijst naar de nieuwe URL.Nog te doen door een mens, na de merge: de oude
pages-branch (Forgejo-restant) opruimen.🤖 Generated with Claude Code