Strongly recommend we fail builds on Checkstyle failures. There's hardly a point to style rules that are not enforced. Additionally, I would argue this makes it _easier_ for contributors to submit code. We frequently critique pull requests based on style conformity, and this would allow contributors to get it right before submitting a pull.