Page MenuHomePhabricator

Update documentation for text linter
AbandonedPublic

Authored by nfirmani on Aug 26 2016, 8:05 PM.
Tags
None
Referenced Files
Unknown Object (File)
Apr 15 2017, 3:31 PM
Unknown Object (File)
Mar 27 2017, 3:45 PM
Unknown Object (File)
Mar 2 2017, 2:30 AM
Unknown Object (File)
Feb 18 2017, 4:56 PM
Unknown Object (File)
Feb 7 2017, 7:37 AM
Unknown Object (File)
Feb 4 2017, 2:18 AM
Unknown Object (File)
Jan 23 2017, 5:09 AM
Unknown Object (File)
Jan 14 2017, 6:36 PM
Subscribers
Tokens
"Baby Tequila" token, awarded by amu.

Details

Summary

Resolves T11520

Test Plan

Previewed the file in an editor window to make sure it looks good.

Diff Detail

Repository
rP Phabricator
Branch
nf_fix_lint_docs_for_text_severity
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 13471
Build 17320: arc lint + arc unit

Event Timeline

nfirmani retitled this revision from to Update documentation for text linter.
nfirmani updated this object.
nfirmani edited the test plan for this revision. (Show Details)
nfirmani added a reviewer: epriestley.

I think a better fix here is to accept either "TXT3" or "3" as keys. I'd rather do that than just document the current not-so-good behavior.

Generally, see T10038 for discussion. I think T3914 has some discussion of this issue in particular, although it is quite old.

Offhand, T8474, T9626, T4287 are also somewhat related.

In a vacuum, this patch is fine, but in the long term it's something we'd likely have to undo when we improve linter behavior. I don't plan to tackle any of this until the next major iteration on lint, which will probably occur after T4287/T11429 and alongside T5055 coming online.

The simplest bugs are always the most complicated.. However, judging by the number of tickets about this issue (T3914, the ones referenced in T8474, T11520), perhaps it's worth the cost of reverting this documentation later vs handling more support tickets. I'll leave that up to you though!