Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Trailing tabs and/or spaces #38

Closed
WestLangley opened this issue Jun 20, 2015 · 2 comments
Closed

Trailing tabs and/or spaces #38

WestLangley opened this issue Jun 20, 2015 · 2 comments
Labels
enhancement

Comments

@WestLangley
Copy link
Collaborator

Trailing tabs and/or spaces should be removed from any line.
@zz85
Copy link
Owner

zz85 commented Jun 21, 2015

i'm planning to use disallowTrailingWhitespace in jscs for this, however it doesn't support auto-fixing, so I'm working on a patch in jscs.

zz85 added a commit to zz85/node-jscs that referenced this issue Jun 23, 2015
@zz85
Support autofix for disallow-trailing-whitespace
e4d8683 
- uses AST tokens by lines instead of just line strings
- add disallow-trailing-whitespace rule to mdcs preset
- fixes zz85/mrdoobapproves#38
@zz85 zz85 mentioned this issue Jun 23, 2015
Closed
zz85 added a commit to zz85/node-jscs that referenced this issue Jun 23, 2015
@zz85
Support autofix for disallow-trailing-whitespace
cfb5466 
- uses AST tokens by lines instead of just line strings
- uses errors.assert to emit errors
- add disallowTrailingWhitespace rule to mdcs preset
- fixes zz85/mrdoobapproves#38
zz85 added a commit to zz85/node-jscs that referenced this issue Jun 23, 2015
@zz85
Support autofix for disallow-trailing-whitespace
d1fe2b0 
- uses AST tokens by lines instead of just line strings
- uses errors.assert to emit errors
- add disallowTrailingWhitespace rule to mdcs preset
- fixes zz85/mrdoobapproves#38
zz85 added a commit to zz85/node-jscs that referenced this issue Jun 23, 2015
@zz85
Support autofix for disallow-trailing-whitespace
53d01f3 
- uses AST tokens by lines instead of just line strings
- uses errors.assert to emit errors
- add disallowTrailingWhitespace rule to mdcs preset
- fixes zz85/mrdoobapproves#38
This was referenced Jun 23, 2015
Closed
Closed
Closed
zz85 added a commit that referenced this issue Jun 26, 2015
@zz85
Add preprocess() and postprocess() for additional processing
9fa19a6 
 before/after jscs

- allows block indentation while maintaining accurate error columns
- remove trailing whitespaces when autoformatting
- fixes #27, #37, #38
@zz85 zz85 mentioned this issue Jun 26, 2015
Merged
@zz85
Copy link
Owner

zz85 commented Aug 18, 2015

closed with #44 and when i bump jscs to version 2.0

@zz85 zz85 closed this as completed Aug 18, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Support autofix for disallow-trailing-whitespace zz85/node-jscs
2 participants
@zz85 @WestLangley