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)
Mon, Mar 25, 10:13 AM
Unknown Object (File)
Mon, Mar 18, 10:43 AM
Unknown Object (File)
Tue, Mar 12, 1:16 AM
Unknown Object (File)
Tue, Mar 12, 1:16 AM
Unknown Object (File)
Feb 15 2024, 12:55 AM
Unknown Object (File)
Feb 15 2024, 12:55 AM
Unknown Object (File)
Feb 15 2024, 12:55 AM
Unknown Object (File)
Feb 15 2024, 12:08 AM

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!