alaazbair commented on issue #25682: [SPARK-28842][DOC]Cleanup the 
formatting/trailing spaces in the K8s integration testing guide
URL: https://github.com/apache/spark/pull/25682#issuecomment-532165439
 
 
   Hi @srowen @dongjoon-hyun , sorry for the late response. I think this change 
will matter the most when between commits we get diffs polluted with unchanged 
lines (during a change in the documentation, an existing trailing space was 
removed ). In addition trailing spaces could break some text editors for 
instance pressing END when the cursor is on a line containing trailing spaces, 
will not move the cursor to the end of the line and this can be annoying if the 
formatting isn't consistent throughout the file.
   
   Besides this I can't think of any other justification. if you think this 
change isn't justified, I will gladly close the PR.
   
   Cheers.

----------------------------------------------------------------
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:
[email protected]


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to