On 18 March 2018 at 09:49, Andrea Aime <andrea.a...@geo-solutions.it> wrote:

> Hi,
> given the issues on the intellij + checkstyle approach I had a second look
> at the google variant
> and cleaned up the branch to have a good separation of pom changes and
> reformats:
>
> https://github.com/aaime/geotools/tree/intellij_checkstyle2
>
> Indeed, it took 3 runs to get to a stable formatting, you can see them as
> separate commits and check
> the changes occurred in each:
> https://github.com/aaime/geotools/tree/google_format
>
> Looking at the code, I'm suffering from mild nausea... that said, the same
> happened when I switched
> from Eclipse to IntelliJ a few months ago, was sick for a week but then it
> went away.
>
>
>From the quick look I've had it isn't too bad, I quite like 2 spaces per
tab (I used it for local projects for a while).

TBH I'm less concerned about what it looks like and more concerned that it
is maintainable going forward.


Thanks for experimenting with this.


Ian
------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
GeoTools-Devel mailing list
GeoTools-Devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geotools-devel

Reply via email to