Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Beheer: voeg bijlage toe over commit message structuur #3

Merged
merged 2 commits into from
Jan 27, 2025

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.
bijlage02_GebruikGithub.md Outdated Show resolved Hide resolved
bijlage02_GebruikGithub.md Show resolved Hide resolved
@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