Add note to Whitespace section on removing trailing whitespace.#163
Add note to Whitespace section on removing trailing whitespace.#163unkillbob wants to merge 1 commit into
Conversation
There was a problem hiding this comment.
This is a good call. It would be the most excellent if we could also include guides on how to set up automatic trailing space removal in various IDEs/text editors.
There was a problem hiding this comment.
The tools exist although they're not as comprehensive:
https://github.com/jshint/fixmyjs
https://github.com/millermedeiros/esformatter
There was a problem hiding this comment.
Sublime Text has the trim_trailing_white_space_on_save setting for this.
There was a problem hiding this comment.
Yeah sublime is
Preferences / Settings - User" to add the following:
{ "trim_trailing_white_space_on_save": true }
IntelliJ under preferences, editor
https://i.imgur.com/JDKfldN.png
JSCS has autofix (jscs src --fix) as of 1.12 and working on applying it to all rules.
|
👍 |
|
@unkillbob it'd be great to land this PR - would you rebase it on top of the latest master, and add a new subsection under "Whitespace" that links to this section? In addition, could you link to the eslint rule for this? |
ljharb
left a comment
ljharb
left a comment
There was a problem hiding this comment.
rebased this as an example in the now-existing trailing whitespace section
|
@unkillbob actually, since you made this PR from a fork you don't own, you'd need to add me to the orionhealth repo for me to be able to land this. |
7 participants
No description provided.