[GNC-dev] merging failing tests

[Geert Janssens]

Op woensdag 5 september 2018 23:12:31 CEST schreef Carsten Rinke:
> Why not merging a failing test? Why waiting for the code fix?

Aside from John's explanation I would add this as a reason:
Maint and master are integration branches. Our intention is to keep them in a 
releasable state all the time. That is at any point in time we should be able 
to take their current state and release it.

I wrote "intention" because in practice new commits can always introduce bugs 
that are not immediately detected. Especially as our test coverage is far from 
complete. So we are not there yet. But the intention is there and a branch 
with failing tests is not in a releasable state. So failing tests are ok on 
feature branches but they should be fixed (or dropped/disabled if irrelevant 
or not critical) before they can be merged.

Geert