This is still a WIP. I would also like to add autofixing to this (although this may be difficult) and also add some whitelisted constructs, such as for loops. Just seeking some initial feedback.
Details
Details
Diff Detail
Diff Detail
- Repository
- rARC Arcanist
- Branch
- master
- Lint
Lint Passed - Unit
Test Failures - Build Status
Buildable 4052 Build 4065: [Placeholder Plan] Wait for 30 Seconds
| Time | Test | |
|---|---|---|
| 4,615 ms | ArcanistXHPASTLinterTestCase::testLinter | |
| 672 ms | ArcanistCSSLintLinterTestCase::testLinter | |
| 126 ms | ArcanistCSSLintLinterTestCase::testVersion | |
| 186 ms | ArcanistChmodLinterTestCase::testLinter | |
| 222 ms | ArcanistClosureLinterTestCase::testLinter | |
| View Full Test Results (1 Failed · 38 Passed · 12 Skipped) |
Event Timeline
Comment Actions
Hmm, I wonder if looking for n_STATEMENT or whatever might be cleaner? Not sure if that would get you everything you want, though.