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
- Lint
Lint Not Applicable - Unit
Tests Not Applicable