[hibernate-dev] Checkstyle: enforced!

Steve Ebersole steve at hibernate.org
Thu Apr 18 10:51:55 EDT 2013


Nice!

Curious how you got checkstyle to validate that tabs are used for indent?
On Apr 18, 2013 9:49 AM, "Sanne Grinovero" <sanne at hibernate.org> wrote:

> Hi all,
>
> yesterday we merged checkstyle integration in Hibernate OGM: it's
> going to fail the compile phase if there are violations (!).
>
> By doing so, we also had to re-style a large portion of code, as
> surprisingly even though I though we where very careful on style and
> whitespace inspections, human eye is still far behind the power of
> such a tool. Sorry for the conflicts this caused: we hope it's a
> smaller price to pay compared to the long term benefits.
>
> I realize this might look very inconvenient initially to have it fail
> during some quick experimental hacking, but the purpose is to
> encourage applying the correct autoformatting styles in your IDE: it
> should not annoy you once you get use to obey the rules.
>
> The purpose is to reduce the back-and-forth discussions for which pull
> requests have been delayed for style reasons, not using tabs for
> indent, missing/wrong copyright headers...
> also it should reduce conflicts in the future.
>
> This definitely will look annoying initially, but please try surviging
> the initial push back and fix your style.
> If it still is annoying in the long run we can discuss relaxing the
> rules or demoting it from a compile failure to some warning levels. If
> things don't look too bad (i.e. I don't get enough complaints) I will
> soon send a pull request for Search too, and I hope other projects
> will follow.
>
> Cheers,
> Sanne
> _______________________________________________
> hibernate-dev mailing list
> hibernate-dev at lists.jboss.org
> https://lists.jboss.org/mailman/listinfo/hibernate-dev
>


More information about the hibernate-dev mailing list