smartsheet / smartsheet-java-sdk

Library that uses Java to connect to Smartsheet services.
Apache License 2.0
3 stars 12 forks source link

java.header has a weird whitespace typo that is "fixed" by IntelliJ's formatter #96

Open ronreynolds opened 1 month ago

ronreynolds commented 1 month ago

i repeatedly have to remove the first padding whitespace (2nd line) in config/checkstyle/java.header which is a real time-suck (because i like using IntelliJ's code-formatter for all the other wonderful things it does). i don't expect there to be much objection but just wanted to create a issue/task for this to add the padding whitespace to line 2 so that post-formatter the java header doesn't fail check-style (which is requiring this exact header appear at the top of every single java file, including unit-tests). actually i think the easiest way to do this is to just apply the code-formatter to every single java file (otherwise there are 356 main and 134 unit-test java files to be updated). unfortunately that's going to be a huge mass of non-functional, mostly white-space, changes.

ronreynolds commented 1 month ago

fix in https://github.com/smartsheet/smartsheet-java-sdk/pull/97