Page MenuHomePhabricator

When lint identifies an unknown symbol, attempt to correct it if it is miscapitalized
ClosedPublic

Authored by epriestley on Feb 3 2021, 4:54 PM.
Tags
None
Referenced Files
F14094561: D21537.id51269.diff
Mon, Nov 25, 4:17 PM
Unknown Object (File)
Sat, Nov 23, 9:36 PM
Unknown Object (File)
Fri, Nov 22, 9:54 AM
Unknown Object (File)
Fri, Nov 22, 4:26 AM
Unknown Object (File)
Wed, Nov 20, 10:04 PM
Unknown Object (File)
Sat, Nov 9, 4:25 PM
Unknown Object (File)
Thu, Oct 31, 12:12 PM
Unknown Object (File)
Oct 17 2024, 2:38 PM
Subscribers
None

Details

Summary

Ref T13598. If you spell a symbol like "Polygon" as "PoLyGoN", you currently get an "unknown symbol" lint message. However, provided "Polygon" is a valid symbol, we can unambiguously correct the spelling of the symbol.

Note that this patch can only correct the spelling of application symbols, not builtin symbols (since none of the library maps contain builtin symbols).

Test Plan

Screen Shot 2021-02-03 at 8.50.07 AM.png (931×1 px, 388 KB)

Diff Detail

Repository
rARC Arcanist
Branch
lint1
Lint
Lint Passed
Unit
Test Failures
Build Status
Buildable 25079
Build 34604: Run Core Tests
Build 34603: arc lint + arc unit

Unit TestsFailed

TimeTest
86 msArcanistRubyLinterTestCase::testLinter
Lint emitted an unexpected set of messages for file "wvswc.lint-test". MISSING MESSAGES (No messages.)
0 msAbstractDirectedGraphTestCase::testCyclicGraph
1 assertion(s) passed.
7 msAbstractDirectedGraphTestCase::testEdgeLoadFailure
1 assertion(s) passed.
0 msAbstractDirectedGraphTestCase::testNonTreeGraph
1 assertion(s) passed.
0 msAbstractDirectedGraphTestCase::testNoncyclicGraph
1 assertion(s) passed.
View Full Test Results (1 Failed · 494 Passed · 65 Skipped)