Re: [DISCUSS] Proposed formatter change: 100 char lines

2018-02-16 Thread Christopher
FWIW, we use the animal-sniffer plugin to verify full Java 7 compatibility to avoid those historical issues with simply setting the target JDK. But, I also understand if that plugin does not instill sufficient confidence for Java 7 compatibility. I can do the format-only-in-jdk8-profile strategy,

Re: [DISCUSS] Proposed formatter change: 100 char lines

2018-02-16 Thread Josh Elser
+1 to not changing min-Java on the release lines that supported Java 7. Let's just cease activity on these branches instead :) On 2/16/18 9:55 AM, Sean Busbey wrote: I'm opposed to requiring Java 8 to build on branches that we claim support running under Java 7. Historically relying on

Re: [DISCUSS] Proposed formatter change: 100 char lines

2018-02-16 Thread Sean Busbey
I'm opposed to requiring Java 8 to build on branches that we claim support running under Java 7. Historically relying on "compile for earlier target JDK" has just led to pain down the road when it inevitably doesn't work. Just make it a recommendation for contributions and have our precommit

Re: [DISCUSS] Proposed formatter change: 100 char lines

2018-02-16 Thread Mike Walch
> To: "accumulo-dev" <dev@accumulo.apache.org> > Sent: 2/15/2018 11:24:31 PM > Subject: [DISCUSS] Proposed formatter change: 100 char lines > > Primarily for accessibility reasons (screen space with a comfortable font), >> but also to support readability for dev

Re: [DISCUSS] Proposed formatter change: 100 char lines

2018-02-16 Thread J. Mark Owens
..@apache.org> To: "accumulo-dev" <dev@accumulo.apache.org> Sent: 2/15/2018 11:24:31 PM Subject: [DISCUSS] Proposed formatter change: 100 char lines Primarily for accessibility reasons (screen space with a comfortable font), but also to support readability for devs working on sensibly-

[DISCUSS] Proposed formatter change: 100 char lines

2018-02-15 Thread Christopher
Primarily for accessibility reasons (screen space with a comfortable font), but also to support readability for devs working on sensibly-sized screens, I want to change our formatter to format with 100 char line length instead of its current 160. Many of our files need to be reformatted anyway,