+1 On Wed, Jan 7, 2015 at 3:12 PM, Christopher <ctubb...@apache.org> wrote:
> To make it easier to apply some minimal checkstyle rules for ACCUMULO-3451, > I'm announcing my intentions to do a full, one-time, auto-format and > organize imports on all our supported branches (1.5, 1.6, and master) to > bring us up to some degree of compliance with our agreed-upon formatting > standards. > > Benefits: > To have additional checks, in particular against javadoc problems and other > common trivial warnings in the build. > To ensure less divergence from our agreed-upon formatting standards. > Formatting first makes it much less tedious and easier on me to add these > checks to the build. > > Issues I've considered: > I will deal with all the merge conflicts. > I will ignore generated thrift code. > Conflicts with new code in people's branches should be minimal (and easily > resolved by formatting according to our standards). > Regarding concerns about history tracking, in general, each format change > is small, but they are numerous. So, the impact on tracking history should > be very minimal (you'll see things like a brace moved to the same line as > the else statement it is associated with... stuff that won't generally > affect your ability to debug). > I'll also do a "format only" commit, separately from any substantive > changes regarding the rule changes, so the mass formatting change will > happen in one place, and it will also be easy to revert, if absolutely > necessary. > > I'll give this 24 hours (it can be reverted if somebody objects after > that). > > -- > Christopher L Tubbs II > http://gravatar.com/ctubbsii >