This removes the consistency_fail enforcer when in development, making it only run when in test mode.
If I understand the intention of this tool correctly we will still get all the benefits of consistency_fail by only enforcing checks in the test env, and not have to deal with the inconvenience of having it in the development env.
I find it makes evaluating a WIP PR (where index are maybe not added yet) much more of a PITA, and given that we will never let a failing branch of code get into master it seems redundant anyway.
This removes the consistency_fail enforcer when in development, making it only run when in test mode.
If I understand the intention of this tool correctly we will still get all the benefits of consistency_fail by only enforcing checks in the
test
env, and not have to deal with the inconvenience of having it in thedevelopment
env.I find it makes evaluating a WIP PR (where index are maybe not added yet) much more of a PITA, and given that we will never let a failing branch of code get into master it seems redundant anyway.
Thoughts?