| 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