Ref T2039. Addresses two issues:
- Issues a warning for use of config which is deprecated by .arclint.
- We no longer require an engine to be present for these linters, so arc linters doesn't fatal if they aren't configured.
Details
Details
- Reviewers
joshuaspence btrahan chad - Maniphest Tasks
- Restricted Maniphest Task
- Commits
- rARC7870e7f2e46d: Warn when accessing deprecated lint config
- Ran arc linters in a repo with no JSHint.
- Ran arc linters in a repo with junk in .arcconfig and got a warning when it was read. Verified it still took effect.
Diff Detail
Diff Detail
- Repository
- rARC Arcanist
- Branch
- lintdeprecate
- Lint
Lint Passed - Unit
Test Failures - Build Status
Buildable 370 Build 370: [Placeholder Plan] Wait for 30 Seconds
| Time | Test | |
|---|---|---|
| 0 ms | testFlake8Lint | |
| 0 ms | testJSHintLinter | |
| 0 ms | testCSSLintLinter | |
| 0 ms | testFixLetterCase | |
| 0 ms | testLintPath | |
| View Full Test Results (2 Failed · 11 Passed · 4 Skipped) |