I think we can work on this one...
First of all - contributors need to know about this rule, perhaps
updating [1] might be a good idea. Official announcement on mailing list
might be also helpful (this email thread is already pretty long, so it
might be missed by many folks).
Secondly - we need to stop integrating Pull Requests with new failures.
It's a bit harder when we have some existing failures, because there is
always an excuse (this failure is not related, it's just an unstable
test etc). But once we have clean build - it's a "binary" decision.
I think we might also add some descriptive comment to Pull Request when
the build is unstable - something like "This Pull Request won't be
integrated, because it's unstable. Fix it first.".
[1]
http://infinispan.org/docs/7.0.x/contributing/contributing.html
On 10/21/2014 10:27 AM, Sanne Grinovero wrote:
I totally agree here, but it never worked: people regularly ignore
failing tests, for various reasons.
We've had similar good intentions expressed many times, but I simply
have no reason to believe that this time it's going to work out.