Remove lf from NewlineAtEndOfFileCheck

[artembilan]

Different OSs may check out files with different line separators.
So, to better support other development environments it would be great to just require any end of file, not just Linux-specific.

[spencergibb]

Not sure about this one. Are you the one having the issue?

[artembilan]

I am, Spencer. I develop on Windows and my Git client checks out files with crlf, so when I try to build Spring Cloud projects, the Checkstyle fails for me that it expects only lf.
It worked in the past because they really checked for end of line, which suited to crlf combination with just that lf config.
But now it fails because the rule has been tightened a little bit.

See similar config in SF: https://github.com/spring-projects/spring-framework/blob/main/src/checkstyle/checkstyle.xml#L16.
The respective change on their side: spring-projects/spring-framework@2feedb9.

We do the same in Spring Integration, Spring AMQP, Spring for Apache Kafka.

See also this for more justifications: spring-io/spring-javaformat#202

[spencergibb]

Thanks for the justifications.

[spencergibb]

Cherry picked to 3.1.x and merged forward via 711fbce