I think that this output was used during the early stage of ArcanistConfigurationDrivenLintEngine, but I question it's value nowadays. In particular, I find that this output makes the output of arc lint --trace significantly less useful.
Details
Details
- Reviewers
epriestley - Group Reviewers
Blessed Reviewers - Commits
- rARC999eb9376568: Remove verbose output from `arc lint --trace`
Ran ./bin/arc lint --trace and saw useful output.
Diff Detail
Diff Detail
- Repository
- rARC Arcanist
- Branch
- master
- Lint
Lint Passed - Unit
Tests Passed - Build Status
Buildable 7199 Build 7442: [Placeholder Plan] Wait for 30 Seconds Build 7441: arc lint + arc unit
Event Timeline
Comment Actions
I expected that users might find these include/exclude rules difficult to understand/debug, but I haven't actually seen any issues. It's vaguely possible that I haven't seen issues because we have this diagnostic output, but I'd expect most users having trouble to not make it as far as --trace, so I think this is probably low value / find to remove.