Currently, a lot of ArcanistExternalLinter subclasses have something like this:
if ($err && !$messages) { return false; }
We can avoid this code duplication by moving this check to the ArcanistExternalLinter class.
Details
Details
- Reviewers
epriestley - Group Reviewers
Blessed Reviewers - Commits
- rARC8244b3582acc: Improve behavior for detecting the failure to parse external linter output
arc unit
Diff Detail
Diff Detail
- Repository
- rARC Arcanist
- Branch
- master
- Lint
Lint Passed - Unit
Tests Passed - Build Status
Buildable 6198 Build 6219: [Placeholder Plan] Wait for 30 Seconds