Change Details

[[ https://secure.phabricator.com/diffusion/ARC/browse/master/src/lint/linter/ArcanistTextLinter.php | ArcanistTextLinter ]] checks for trailing whitespace and trims it. This is suboptimal solution for these reasons: # It asks user whether it should apply the changes by default. # In `arc diff`, we don't start unit tests until lint finishes. Optimal solution is to strip the trailing whitespace in editor upon saving a file. Documentation explaining how to set this up in different editors may be useful. We are interested at least in these editors: * vi * emacs * Eclipse * Xcode Please include screenshots.

[[ https://secure.phabricator.com/diffusion/ARC/browse/master/src/lint/linter/ArcanistTextLinter.php | ArcanistTextLinter ]] checks for trailing whitespace and trims it. This is suboptimal solution for these reasons: # It asks user whether it should apply the changes by default. # In `arc diff`, we don't start unit tests until lint finishes. Optimal solution is to strip the trailing whitespace in editor upon saving a file. Documentation explaining how to set this up in different editors may be useful. We are interested at least in these editors: * vi * emacs * Eclipse * Xcode Please include screenshots and link this document from the lint message.

[[ https://secure.phabricator.com/diffusion/ARC/browse/master/src/lint/linter/ArcanistTextLinter.php | ArcanistTextLinter ]] checks for trailing whitespace and trims it. This is suboptimal solution for these reasons: # It asks user whether it should apply the changes by default. # In `arc diff`, we don't start unit tests until lint finishes. Optimal solution is to strip the trailing whitespace in editor upon saving a file. Documentation explaining how to set this up in different editors may be useful. We are interested at least in these editors: * vi * emacs * Eclipse * Xcode Please include screenshots and link this document from the lint message.