chore: pre-commit-hookbron naar GitHub - #4
Open
MWest2020 wants to merge 2 commits into
Open
Conversation
De techbook-hook kwam van codeberg.org/Conduction/techbook. Dat was de laatste harde Codeberg-afhankelijkheid buiten talos: zolang die bestond moest techbook naar twee forges gepusht blijven worden, en dat is niet volgehouden — 7 van de 9 repos zijn daar uit elkaar gelopen. De pin edf269ee blijft ongewijzigd: geverifieerd dat die commit op GitHub bestaat en daar voorouder van main is. Host-only omzetting dus, geen gedragswijziging in de gates. Geverifieerd: pre-commit install-hooks resolvet de hook van GitHub (exit 0) en docs-contract passeert in de pre-push-stage. Ook nagegaan dat github.com al in de squid-allowlist van talos staat (base en con-ci-oci), dus CI achter default-deny egress kan de hooks blijven ophalen. De bron van het patroon zat in techbook/scripts/rollout_precommit_hook.sh, dat deze URL in elke repo schreef; die is in dezelfde ronde omgezet. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
…github # Conflicts: # CHANGELOG.md
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.
Eén regel: de techbook-hook komt van
github.com/ConductionNL/techbookin plaats vancodeberg.org/Conduction/techbook.Waarom
Dit was de laatste harde Codeberg-afhankelijkheid in de fleet buiten talos. Zolang die bestond, moest
techbooknaar twee forges gepusht blijven worden — en dat is aantoonbaar niet volgehouden: 7 van de 9 repos zijn op Codeberg uit elkaar gelopen met GitHub.Waarom dit veilig is
De pin
edf269ee…blijft ongewijzigd. Geverifieerd dat die commit op GitHub bestaat én daar voorouder vanmainis, met dezelfde hooks (docs-contract,docs-claims). Dit is dus een host-omzetting, geen gedragswijziging in de gates.Verificatie
Niet alleen de string vervangen — functioneel doorgetest:
pre-commit install-hooksresolvet de hook van GitHub, exit 0pre-commit run docs-contract --all-files --hook-stage pre-push→ Passedgithub.comstaat al in de squid-allowlist van talos (base éncon-ci-oci-overlay), dus CI achter default-deny egress kan de hooks blijven ophalen — geen egress-wijziging nodigHoort bij
ConductionNL/techbook — daar zat de oorzaak:
scripts/rollout_precommit_hook.shschreef deze URL in élke repo, dus elke nieuwe deelnemer kreeg de Codeberg-afhankelijkheid er automatisch bij. Zelfde patroon alsclone_all.shin hub. Die PR zetHOOK_REPO_URLom (env-tunable) en voegt een tak toe die tijdens de migratie zegt "entry staat nog op codeberg.org" in plaats van misleidend "voeg de hook handmatig toe".Zonder die techbook-PR schrijft het uitrolscript de oude URL terug bij de volgende deelnemer.
🤖 Generated with Claude Code