Here's another idea to mitigate this. The problem here is "a broken build", which really in most cases the broken build is really just a lint issue related to a bad merge. Maybe the solution is that we should not break the build on lint errors, and instead warn on the PR, and have a policy (either hard or softly enforced) to not merge PRs with lint.
This way, when lint shows up as a byproduct or a merge, it wouldn't break the build, it would just generate a tiny bit of lint for anyone who rebases while in this state, and those who do would fix that one-liner. That would prevent the false-negative "broken build". [ Full content available at: https://github.com/apache/incubator-superset/pull/5918 ] This message was relayed via gitbox.apache.org for [email protected]
