| 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