>> | I understand the concern, but if we're creating a world >> | where tests have to wait for developer review, we're doing >> | CI the wrong way around. The whole point of automated >> | testing is to minimize the need for human review, so we >> | should aim to run as many tests as possible as early as >> | possible. Both test performance and security considerations >> | are problems to be gradually resolved in aiming for highest >> | possible test execution for all code that gets submitted, >> | and minimizing the need for human review on code which is >> | obviously broken. >> >> Why not just add (more) slaves? Computing power is much >> cheaper than developer time. > >I absolutely agree.
I'm also in agreement on this one. Having tests run after code review negates the point of having tests run in the first place. Thank you, Derric Atzrott _______________________________________________ Wikitech-l mailing list Wikitech-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikitech-l
