Skip to content

Beheer: voeg bijlage toe over commit message structuur#3

Merged
TimvdLippe merged 2 commits intodevelopfrom
pr/timvdlippe/commit-message-guidelines
Jan 27, 2025
Merged

Beheer: voeg bijlage toe over commit message structuur#3
TimvdLippe merged 2 commits intodevelopfrom
pr/timvdlippe/commit-message-guidelines

Conversation

@TimvdLippe
Copy link
Contributor

Hiermee verduidelijken we wat het formaat en structuur
van een commit message is en dat die in het Nederlands
moeten zijn.

Hiermee verduidelijken we wat het formaat en structuur
van een commit message is en dat die in het Nederlands
moeten zijn.
@sanderke
Copy link
Member

Mee eens meer aandacht te besteden aan commit messages; Vooral bij blame zou het handig zijn om te zien uit welk overleg o.i.d. een wijziging voort kwam.
Wanneer iemand in een branch bezig hoeft men niet per commit te peinzen en pas bij een (squash en) merge wordt een goede omschrijving gedicht.

@TimvdLippe
Copy link
Contributor Author

TimvdLippe commented Jan 23, 2025

@TimvdLippe TimvdLippe changed the title Editorial: voeg bijlage toe over commit message structuur Beheer: voeg bijlage toe over commit message structuur Jan 24, 2025
Copy link

@PHaasnoot PHaasnoot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Goed voorstel, nog te bespreken of dit in het Beheermodel moet of in aparte generieke github werkwijze beschrijving, immers API-Standaarden Beheermodel valt onder MIDO governance , API Beheermodel is (en wordt) vastgesteld door programmeringsraad gdi,

@TimvdLippe
Copy link
Contributor Author

Overlegd binnen het team en er is akkoord. Wat betreft de huidige locatie van de GitHub werkwijze als bijlage van het beheermodel gaan we in de toekomst kijken of dit beter zou passen in een CONTRIBUTING.md. Dat nieuwe bestandje is een industry-standard en ook voor externen makkelijker te vinden. Hierin kan dan de volledige bijlage in worden verwerkt. Echter, omdat dat een bredere discussie is over de hele bijlage en niet deze ene sectie, schuiven we dat door naar een apart overleg.

@TimvdLippe TimvdLippe merged commit cb7db68 into develop Jan 27, 2025
5 checks passed
@TimvdLippe TimvdLippe deleted the pr/timvdlippe/commit-message-guidelines branch January 27, 2025 15:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants