-
Notifications
You must be signed in to change notification settings - Fork 13
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
docs: definition of done improvements #775
base: main
Are you sure you want to change the base?
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
zou twee nog iets breder maken zodat er eerder aan kan worden voldaan, en eentje weglaten
- Component is publiek beschikbaar. | ||
- Code is publiek beschikbaar via Git. | ||
- CSS is gepubliceerd in een publieke NPM package. | ||
- Tellen welke varianten veel gebruikt worden. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
‘veel’ is niet concreet, dit item lijkt me sowieso geen geschikt DoD item, zou 'm eruit halen
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Het doel is "snowflake" varianten eruit halen, dus wel waardevol. Ik zal 'm iets aanscherpen.
Co-authored-by: Hidde de Vries <hidde@hiddedevries.nl>
Co-authored-by: Hidde de Vries <hidde@hiddedevries.nl>
No description provided.