Page MenuHomePhabricator

Removed useless comments.
ClosedPublic

Authored by joshuaspence on Mar 2 2014, 3:10 AM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Dec 17, 6:20 AM
Unknown Object (File)
Thu, Nov 28, 6:33 PM
Unknown Object (File)
Thu, Nov 28, 6:33 PM
Unknown Object (File)
Thu, Nov 28, 6:33 PM
Unknown Object (File)
Thu, Nov 28, 6:14 PM
Unknown Object (File)
Wed, Nov 27, 12:23 AM
Unknown Object (File)
Wed, Nov 27, 12:23 AM
Unknown Object (File)
Wed, Nov 27, 12:23 AM

Details

Summary

To me, it seems that these comments add no value. Personal opinion I suppose.

Test Plan

N/A

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

epriestley edited edge metadata.

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.

This revision is now accepted and ready to land.Mar 4 2014, 6:52 PM