I feel that we are abusing ArcanistUsageException. Throw a more tailed exception instead. Depends on D11197.
Details
Details
- Reviewers
epriestley - Group Reviewers
Blessed Reviewers - Commits
- rARC6eed5c251411: Create a custom exception class for missing linter dependencies
arc lint, I guess.
Diff Detail
Diff Detail
- Repository
- rARC Arcanist
- Lint
Lint Not Applicable - Unit
Tests Not Applicable
Event Timeline
Comment Actions
Oh, yeah, I probably wouldn't extend.
Any exception could technically be considered the result of someone misusing something, but I think that distinction becomes meaningless long before we get into a major configuration issue like missing software dependencies.