Page MenuHomePhabricator

Added `ArcanistTextLinter::LINT_BOF_WHITESPACE` and `ArcanistTextLinter::LINT_EOF_WHITESPACE`
ClosedPublic

Authored by joshuaspence on Jan 14 2014, 1:41 AM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Nov 20, 11:19 AM
Unknown Object (File)
Oct 18 2024, 6:27 PM
Unknown Object (File)
Oct 13 2024, 10:15 AM
Unknown Object (File)
Sep 19 2024, 12:20 PM
Unknown Object (File)
Sep 19 2024, 12:20 PM
Unknown Object (File)
Sep 10 2024, 6:02 PM
Unknown Object (File)
Sep 1 2024, 7:31 PM
Unknown Object (File)
Aug 27 2024, 9:33 PM

Details

Summary

This might not be universally desireable, but I found myself writing an additional linter (which I had called WhitespaceTextLinter) for the sake of these two linter tests. I figured it may be of use upstream, and so I decided to submit it as a diff. I won't be offended if it is rejected however.

Test Plan

arc lint and arc unit are both okay with it.

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

joshuaspence updated this revision to Unknown Object (????).Jan 14 2014, 1:54 AM

Used arc lint to demonstrate the changes.

These rules seem reasonable to me, happy to bring them to the upstream. Thanks!