Just my $.02, as a long-time lurker who would love to simplify
contributions as much as possible: I use 2 space indentation in my day job,
and contributing some of that work upstream to you is certainly simpler if
that also uses two spaces ;-]
Best regards,
Eric
On Sun, Mar 18, 2018 at 3:22 AM Ian Turton <ijtur...@gmail.com> wrote:
>
>
> 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
>
------------------------------------------------------------------------------
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
