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
- Branch
- master
- Lint
Lint Passed - Unit
Tests Passed - Build Status
Buildable 3557 Build 3565: [Placeholder Plan] Wait for 30 Seconds
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.