[GitHub] [incubator-nuttx-apps] patacongo commented on issue #56: Ensure all source code end with one and only one newline

[GitBox <gi...@apache.org>]

patacongo commented on issue #56: Ensure all source code end with one and only one newline
URL: https://github.com/apache/incubator-nuttx-apps/pull/56#issuecomment-583403450
 
 
   "is this safe on all types of files?"  It is inappropriate on the .txt file that needs the newline for visual spacing.
   
   Other source files will depend on if there is 0 or 1 newlines at the end.  Looking at the diffs, I cannot tell if there is 0 or 1 newlines at the end now.
   
   I would think that most apps should work with a single newline.  Many applications are even tolerant of missing newlines at the end altogether.  kconfig-frontends is not.  Kconfig files with no newline at the end causes errors.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services