-
Notifications
You must be signed in to change notification settings - Fork 6
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
Update GitHub workflow #148
Conversation
In beginsel akkoord met 1 vraag; ik zou naast de 2 reviewers een extra slag door @praseodym / @jschuurk-kr / als code owners een goed idee vinden. Het idee is dan dat bv 2 mensen binnen het project nog een extra check doen op consistentie, architectuur, kwalteit. Het kan zijn dat het teveel bureaucratie oplevert of gezien de schaal van het team overdreven is. Hoe denken jullie daarover? |
@ring-ring-ring @praseodym Als stap in het proces wordt dat te zwaar. Daarnaast, als we met 2 reviewers geen code kunnen produceren die goed genoeg is, dan hebben we een probleem dat niet opgelost gaat worden door meer reviews toe te voegen. Buiten dat primaire proces om ga ik ervan uit dat Mark, ik, en iedereen die dat wil, regelmatig een stapje terug doet om naar het grotere plaatje van onze code base, applicatie, etc te kijken. |
Eens met @jschuurk-kr. Ik denk ook dat het een vertragende factor zal worden, zeker als een van ons op vakantie/afwezig is. |
Co-authored-by: Joep Schuurkes <145749778+jschuurk-kr@users.noreply.github.com>
Graag ook op- en aanmerkingen over de werkwijze zelf en niet alleen de beschrijving daarvan.