feat(git): een verzegeld deck wordt geweigerd, niet stil uitgekleed (#541) #733
No reviewers
Labels
No labels
accepted
bug
declined
docs
duplicate
enhancement
good first issue
in-progress
needs-info
privacy
security
triage
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set
Reference
LibreKAT/Ocideck!733
Loading…
Reference in a new issue
No description provided.
Delete branch "feat/zegel-naar-git-541"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Deel 3 van #541, volgens ontwerpbesluit D13. Ink blijft liggen — dat begint bij een formaatwijziging met versieverhoging en hoort een eigen ronde te krijgen.
Wat er misging. Een verzegeld deck ging gewoon naar een werkbranch en alleen het zegel viel weg, met een waarschuwing achteraf. Een verzegeld rapport dat via git terugkomt zónder zegel leest als een rapport dat nooit verzegeld is.
Een zegel is een uitspraak over precies díé bytes, en een tak biedt dat niet: die kan herschreven, gecherrypickt en geforceerd geduwd worden. Dus weigeren op het moment dat de keuze er nog is.
De weigering staat in het opslagpad, niet alleen in een dialoog. Dat is de enige plek waar élke aanroeper langskomt; een poort die je omzeilt door een andere knop te gebruiken is geen poort. De interface legt hem uit, met één knop en zonder "toch opslaan".
De waarschuwing krimpt mee, precies zoals het issue vroeg: hij verliest de zegelregel en houdt de tekeningen.
Wat dit níét oplost, en dat staat nu in het ontwerp. D13 noemt twee gevolgen. De weigering is af; de andere helft — "het zegel reist mee in de commitset van een versie-uitgifte" — is niet te bouwen zoals het er staat.
tagReleasetagt de kop van de standaardbranch en commit niets, dus er ís op dat moment geen commitset. De enige weg naar de standaardbranch is een gemergede PR vanaf een werkbranch, en díé weigert dit besluit nu juist.Gevolg: een verzegeld deck kan op dit moment helemaal niet naar de repo. Dat is eerlijker dan het zegel stil laten vallen, maar het is een doodlopende weg. Welke kant open moet is een besluit en geen detail; de drie opties staan uitgeschreven bij D13 in
GIT_STORAGE.md. Tot dan wijst de weigering naar een bestand of een.ocideck-pakket en zegt er met zoveel woorden bij dat de tag-route er nog niet is, in plaats van een weg te beloven die niet bestaat.Bijvangst: twee besluiten deelden het nummer D12 — "How media comes back" en "A seal belongs on a tag". Allebei worden ze elders in het document én vanuit de code aangehaald, dus was onduidelijk welke bedoeld werd. Het jongste is D13 geworden.
Twee ratchets wezen onderweg de weg, en in tegengestelde richting: de methodelengte wilde
_saveToGitgesplitst (het melden van de uitkomst is een ander onderwerp dan het opbouwen van de opslag), het klasseplafond liet zien dat een helper op dezélfde klasse de méthode korter maakt maar de klasse groter. Vandaar dat de weigering een top-level functie in het part-bestand is, net als_reportMissingChartData.TabsNotifierkomt uit op 2211 — nog onder de 2251 van vanochtend.Poort:
make checkgroen, gedraaid ná de rebase op main met #730 erin.Refs #541