issue tempaltes#15
Conversation
MWest2020
left a comment
There was a problem hiding this comment.
Review (own PR — informational)
Templates zelf zijn goed gevormd. Eén waarschijnlijke correctness-bug en wat polish vóór merge.
🟡 Belangrijk — folder-naam
GitHub vereist issue-templates in .github/ISSUE_TEMPLATE/ — dus de .github subfolder van repo root. Voor org-meta-repo's die zelf .github heten kan de bare ISSUE_TEMPLATE/ aan root werken (omdat de repo zelf functioneert als de .github folder voor de org), maar dit is non-standard en heb ik niet zelf geverifieerd.
Test post-merge:
ga naar een willekeurige ConductionNL repo zonder eigen templates, klik "New issue" — als alleen "Open a blank issue" verschijnt was de folder-naam fout en moeten ze naar .github/ISSUE_TEMPLATE/....
🟡 Substantieel — label-bootstrap
PR-beschrijving zegt labels worden auto-applied. Caveat: labels zijn per-repo, niet org-wide in GitHub. Wat dat betekent:
- Labels aanmaken in
ConductionNL/.githubpropageert niet. - Templates die een missing label proberen toe te passen → issue wordt aangemaakt zónder label (geen error, silent skip).
Suggesties:
- Add een idempotent bootstrap-snippet in de PR-beschrijving:
for repo in $(gh repo list ConductionNL --limit 200 --json name -q '.[].name'); do gh label create user-story --color 0075ca --force -R ConductionNL/$repo gh label create needs-refinement --color e4e669 --force -R ConductionNL/$repo gh label create needs-triage --color d93f0b --force -R ConductionNL/$repo gh label create technical --color bfd4f2 --force -R ConductionNL/$repo done
- Of accepteer silent-skip en laat repos backfillen op first use.
🟢 Polish
1. Titel-typo — "issue tempaltes" → suggestie: feat: add org-wide issue templates (conventional commit, geen typo).
2. Severity required, geen "unknown" optie (bug-report.yml). Een melder weet vaak niet of iets kritisch is — dat is triage. Twee opties:
- Maak
required: falsezodat triage het invult, of - Voeg "Unknown / needs triage" als default optie toe.
Anders krijg je severity-inflatie ("alles is critical zodat het wordt gezien").
3. Acceptatiecriteria-shape overlap. Drie templates hebben elk eigen DoD-checklist. "4-eyes review" en "no new security findings" verschijnen in alle drie — kan in een gedeelde intro-block, of accepteer duplicatie. Waarschijnlijk niet de moeite waard om te abstraheren.
4. Overweeg config.yml. .github/ISSUE_TEMPLATE/config.yml met blank_issues_enabled: false forceert template-keuze. Optional maar nuttig als je users door deze templates wilt sturen.
5. Stale-approved sinds 2026-04-12. Remko48 approved bijna een maand terug. Ofwel snel mergen (na folder-check + titel-fix), ofwel sluiten als hij superseded is.
Verdict
Geen approve/request-changes (eigen PR). Aanbeveling: na folder-path verificatie + titel-fix mergen. Label-bootstrap snippet kan als follow-up issue.
Posted as comment — GitHub blokkeert APPROVE op eigen PR's.
Samenvatting
Deze PR voegt drie gestandaardiseerde GitHub Issue templates toe aan de org-level
.githubrepo. Doordat ze hier staan, zijn ze automatisch beschikbaar voor alle ConductionNL-repo's — zonder dat elke repo eigen templates hoeft bij te houden. Repo's die eigen templates nodig hebben kunnen die altijd lokaal overschrijven.Templates
✨ User Story
Voor nieuwe functionaliteit vanuit gebruikersperspectief. Bevat het klassieke Als [rol] wil ik [actie] zodat [waarde] formaat, contextblok, vooraf ingevulde acceptatiecriteria (DoD-checkboxes), een expliciet "buiten scope" veld, prioriteit en optioneel story points.
🐛 Bug Report
Voor het melden van onverwacht gedrag. Bevat reproductiestappen, verwacht vs. werkelijk gedrag, omgevingsinfo (namespace, image tag), ruimte voor logs/screenshots, severity-dropdown en acceptatiecriteria gericht op de fix inclusief regressietest.
⚙️ Technische Taak
Voor infra-werk, refactoring, technische schuld en ops-taken zonder directe gebruikerswaarde. Bevat een aanpakblok, risico's en afhankelijkheden, categorie-dropdown (infra, security, CI/CD, etc.) en acceptatiecriteria die nadruk leggen op auditbaarheid: geen handmatige stappen, rollback beschreven, 4-eyes review.
Acceptatiecriteria
user-story,bug,technical,needs-refinement,needs-triageLabels aanmaken
De templates kennen automatisch labels toe. Die moeten nog bestaan in de org of per repo. Aanmaken via GitHub UI → repo → Issues → Labels, of via de CLI: