HomePhabricator

Add spell check linter
c5dfa34f1055Unpublished

Unpublished Commit · Learn More

No further details are available.

Description

Add spell check linter

Summary:
Inspired by http://news.ycombinator.com/item?id=3464671 and a lot of
diffs I've seen @ FB, I've added a spell checking linter. To reduce
false positives, it's only a blacklist. Still, it catches a large
number of 'issues'.

Test Plan:
Unit tests. Ran on FB's codebase. No false positives
noticed but a lot of cases caught.

Reviewers: epriestley

Reviewed By: epriestley

CC: aran, epriestley, jack

Differential Revision: https://secure.phabricator.com/D1409

Details

Provenance
jackAuthored on
epriestleyCommitted on Jan 16 2012, 5:28 AM
Reviewer
epriestley
Differential Revision
Restricted Differential Revision
Parents
rARC7e63e232ba6c: Fix for `arc install-certificate` timeout.
Branches
Unknown
Tags
Unknown

Event Timeline