To me, it seems that these comments add no value. Personal opinion I suppose.
Details
Details
- Reviewers
epriestley - Group Reviewers
Blessed Reviewers - Commits
- rARC53492fd01e59: Removed useless comments.
N/A
Diff Detail
Diff Detail
- Lint
Lint Skipped - Unit
Tests Skipped
Event Timeline
Comment Actions
I might have given some feedback like "document the options this linter takes", and whoever was contributing the patch did this, and then I missed it on accept. Or I might have just derped this up.
Since I want to deprecate .arcconfig options anyway, removing these fragments seems much cleaner to me. I agree that they're currently pretty useless. Generally, I'd like to push documentation aggressively toward being more terse, particularly after T988, which generally allows documentation to be inherited.
Comment Actions
Closed by commit rARC53492fd01e59 (authored by @joshuaspence, committed by @epriestley).