Page MenuHomePhabricator

Remove `@group` annotations
ClosedPublic

Authored by joshuaspence on Jul 8 2014, 12:59 PM.
Tags
None
Referenced Files
F14078333: D9855.diff
Fri, Nov 22, 3:16 AM
Unknown Object (File)
Wed, Nov 20, 5:14 PM
Unknown Object (File)
Wed, Nov 20, 4:03 AM
Unknown Object (File)
Sat, Nov 2, 7:46 AM
Unknown Object (File)
Wed, Oct 30, 1:18 PM
Unknown Object (File)
Wed, Oct 30, 1:17 PM
Unknown Object (File)
Wed, Oct 30, 1:16 PM
Unknown Object (File)
Wed, Oct 30, 12:52 PM

Details

Summary

I'm pretty sure that @group annotations are useless now... I believe that they were originally used by Diviner?

Test Plan

Eye-balled it.

Diff Detail

Repository
rARC Arcanist
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

joshuaspence retitled this revision from to Remove `@group` annotations.
joshuaspence updated this object.
joshuaspence edited the test plan for this revision. (Show Details)
joshuaspence added a reviewer: epriestley.
epriestley edited edge metadata.

Yeah, Diviner can figure out @group from configuration now. It's still parsed, but only useful to override the defaults if some file has an unusual grouping for some reason (which nothing in these libraries should, I think).

This revision is now accepted and ready to land.Jul 8 2014, 2:42 PM
In D9855#6, @epriestley wrote:

Yeah, Diviner can figure out @group from configuration now. It's still parsed, but only useful to override the defaults if some file has an unusual grouping for some reason (which nothing in these libraries should, I think).

Is there an easy way to identify these?

I don't have any way to identify them if they do exist. They would have to be weird edge cases that I annotated and then forgot about. If there are any, they're so rare that it's not worth trying to find them. We can add them back if necessary later, but more likely we'd just want to move the code now.

The only case like this that I can think of is all the stuff that used to live in libphutil/src/__phutil_library_init__.php -- moving it out was clearly a better approach.