The most important thing I have learned by developing in a quite huge team on the same project over multiple years is that “Too many cooks spoil the broth”.<\/p>\n
Situation<\/strong> Pro: Con: Solution<\/strong> Pro: Con: The most important thing I have learned by developing in a quite huge team on the same project over multiple<\/p>\n","protected":false},"author":1,"featured_media":0,"comment_status":"closed","ping_status":"closed","sticky":false,"template":"","format":"standard","meta":{"inline_featured_image":false},"categories":[38],"tags":[],"_links":{"self":[{"href":"http:\/\/www.max-sperling.bplaced.net\/index.php?rest_route=\/wp\/v2\/posts\/9812"}],"collection":[{"href":"http:\/\/www.max-sperling.bplaced.net\/index.php?rest_route=\/wp\/v2\/posts"}],"about":[{"href":"http:\/\/www.max-sperling.bplaced.net\/index.php?rest_route=\/wp\/v2\/types\/post"}],"author":[{"embeddable":true,"href":"http:\/\/www.max-sperling.bplaced.net\/index.php?rest_route=\/wp\/v2\/users\/1"}],"replies":[{"embeddable":true,"href":"http:\/\/www.max-sperling.bplaced.net\/index.php?rest_route=%2Fwp%2Fv2%2Fcomments&post=9812"}],"version-history":[{"count":1,"href":"http:\/\/www.max-sperling.bplaced.net\/index.php?rest_route=\/wp\/v2\/posts\/9812\/revisions"}],"predecessor-version":[{"id":16771,"href":"http:\/\/www.max-sperling.bplaced.net\/index.php?rest_route=\/wp\/v2\/posts\/9812\/revisions\/16771"}],"wp:attachment":[{"href":"http:\/\/www.max-sperling.bplaced.net\/index.php?rest_route=%2Fwp%2Fv2%2Fmedia&parent=9812"}],"wp:term":[{"taxonomy":"category","embeddable":true,"href":"http:\/\/www.max-sperling.bplaced.net\/index.php?rest_route=%2Fwp%2Fv2%2Fcategories&post=9812"},{"taxonomy":"post_tag","embeddable":true,"href":"http:\/\/www.max-sperling.bplaced.net\/index.php?rest_route=%2Fwp%2Fv2%2Ftags&post=9812"}],"curies":[{"name":"wp","href":"https:\/\/api.w.org\/{rel}","templated":true}]}}
\n– There were no real maintainer of the modules
\n– The developers worked everywhere in the code
\n– Code reviews happened without a real system<\/p>\n
\n– The workload was getting spread evenly
\n– Every developer had a cross module knowledge<\/p>\n
\n– The code was a mess quite quickly and it got worse
\n– Every developer knew just parts of every module<\/p>\n
\n
\n– Assign maintainer for the modules (primary \/ secondary)
\n– Most changes should come from their maintainer if possible
\n– If they haven’t done the changes they should review them<\/p>\n
\n– The code is quite consistent and not such a mess
\n– The maintainer know their modules really good<\/p>\n
\n– The workload isn’t spread evenly all the time
\n– The developers don’t have a cross module knowledge<\/p>\n","protected":false},"excerpt":{"rendered":"