Page MenuHomePhabricator
Differential D9856

Remove `@group` annotations
ClosedPublic
Actions

Authored by joshuaspence on Jul 8 2014, 10:57 PM.
Tags
None
Referenced Files
F17836221: D9856.diff
Sat, Jul 26, 3:55 PM
Unknown Object (File)
Sun, Jun 29, 2:04 AM
Unknown Object (File)
Jun 15 2025, 4:52 AM
Unknown Object (File)
May 25 2025, 9:57 AM
Unknown Object (File)
May 13 2025, 4:13 PM
Unknown Object (File)
Apr 25 2025, 12:15 AM
Unknown Object (File)
Apr 23 2025, 5:40 PM
Unknown Object (File)
Apr 18 2025, 4:56 AM
View All Files
Subscribers
epriestley
Korvin

Details

Reviewers
epriestley
Group Reviewers
Blessed Reviewers
Commits
rPHUed335c485c0a: Remove `@group` annotations
Summary

I'm pretty sure that @group annotations are useless now... see D9855.

Test Plan

Eye-ball it.

Diff Detail

Repository
rPHU libphutil
Branch
rm_group
Lint
Lint Errors
SeverityLocationCodeMessage
Errorsrc/aphront/storage/connection/mysql/AphrontMySQLiDatabaseConnection.php:133XHP45PHP Compatibility
Errorsrc/filesystem/Filesystem.php:558XHP45PHP Compatibility
Errorsrc/future/http/HTTPSFuture.php:533XHP45PHP Compatibility
Advicesrc/future/exec/ExecFuture.php:167XHP16TODO Comment
Unit
Tests Passed
Build Status
Buildable 1579
Build 1580: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

joshuaspence updated this revision to Diff 23649.Jul 8 2014, 10:57 PM
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.
Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptJul 8 2014, 10:57 PM
Herald added subscribers: Korvin, epriestley. · View Herald Transcript
Harbormaster completed remote builds in B1579: Diff 23649.Jul 8 2014, 10:58 PM
epriestley accepted this revision.Jul 9 2014, 10:01 PM
epriestley edited edge metadata.
This revision is now accepted and ready to land.Jul 9 2014, 10:01 PM
joshuaspence closed this revision.Jul 9 2014, 10:08 PM
joshuaspence updated this revision to Diff 23676.

Closed by commit rPHUed335c485c0a (authored by @joshuaspence).

Large Diff

This large diff affects 218 files. Files without inline comments have been collapsed. Expand All Files

Revision Contents
Changeset List

PathSize
src/
3 lines
aphront/
storage/
connection/
4 lines
2 lines
3 lines
mysql/
3 lines
3 lines
2 lines
exception/
5 lines
5 lines
5 lines
5 lines
5 lines
7 lines
3 lines
7 lines
5 lines
3 lines
8 lines
7 lines
writeguard/
3 lines
3 lines
event/
3 lines
cache/
1 line
2 lines
1 line
2 lines
1 line
1 line
1 line
channel/
2 lines
2 lines
2 lines
2 lines
2 lines
2 lines
conduit/
3 lines
3 lines
3 lines
console/
1 line
3 lines
3 lines
3 lines
4 lines
2 lines
__tests__/
4 lines
16 lines
daemon/
1 line
2 lines
torture/
2 lines
2 lines
2 lines
2 lines
2 lines
2 lines
6 lines
error/
1 line
1 line
1 line
2 lines
events/
1 line
3 lines
1 line
constant/
7 lines
1 line
filesystem/
1 line
1 line
1 line
2 lines
1 line
3 lines
3 lines
2 lines
2 lines
__tests__/
38 lines
6 lines
4 lines
10 lines
linesofalarge/
2 lines
7 lines
2 lines
__tests__/
3 lines
3 lines
future/
22 lines
2 lines
5 lines
2 lines
__tests__/
3 lines
asana/
3 lines
aws/
3 lines
3 lines
3 lines
3 lines
exec/
1 line
12 lines
2 lines
__tests__/
8 lines
10 lines
1 line
http/
8 lines
14 lines
4 lines
status/
3 lines
3 lines
3 lines
3 lines
3 lines
paypal/
3 lines
twitch/
3 lines
internationalization/
4 lines
4 lines
__tests__/
2 lines
4 lines
3 lines
4 lines
lexer/
2 lines
4 lines
6 lines
3 lines
3 lines
3 lines
markup/
6 lines
__tests__/
28 lines
3 lines
engine/
__tests__/
2 lines
remarkup/
2 lines
blockrule/
1 line
1 line
3 lines
3 lines
3 lines
4 lines
3 lines
4 lines
12 lines
4 lines
3 lines
3 lines
3 lines
3 lines
markuprule/
2 lines
6 lines
6 lines
6 lines
6 lines
6 lines
6 lines
6 lines
6 lines
27 lines
syntax/
engine/
6 lines
4 lines
__tests__/
2 lines
highlighter/
3 lines
4 lines
3 lines
4 lines
3 lines
3 lines
3 lines
3 lines
6 lines
3 lines
__tests__/
3 lines
3 lines
pygments/
3 lines
xhpast/
3 lines
moduleutils/
3 lines
3 lines
2 lines
9 lines
parser/
4 lines
__tests__/
3 lines
3 lines
3 lines
3 lines
3 lines
3 lines
2 lines
aast/
api/
24 lines
3 lines
4 lines
4 lines
argument/
2 lines
3 lines
__tests__/
3 lines
3 lines
exception/
7 lines
7 lines
7 lines
workflow/
1 line
3 lines
xhpast/
api/
3 lines
3 lines
3 lines
3 lines
__tests__/
4 lines
readableserializer/
9 lines
symbols/
5 lines
exception/
5 lines
utils/
2 lines
__tests__/
4 lines
3 lines
2 lines
4 lines
5 lines
7 lines
64 lines
3 lines
29 lines
57 lines
xsprintf/
3 lines
5 lines
4 lines
6 lines
1 line
15 lines
21 lines
1 line

Diff 23649

View Options

src/__phutil_library_init__.php

View Options

src/aphront/storage/connection/AphrontDatabaseConnection.php

View Options

src/aphront/storage/connection/AphrontDatabaseTransactionState.php

View Options

src/aphront/storage/connection/AphrontIsolatedDatabaseConnection.php

View Options

src/aphront/storage/connection/mysql/AphrontMySQLDatabaseConnection.php

View Options

src/aphront/storage/connection/mysql/AphrontMySQLDatabaseConnectionBase.php

View Options

src/aphront/storage/connection/mysql/AphrontMySQLiDatabaseConnection.php

View Options

src/aphront/storage/exception/AphrontQueryAccessDeniedException.php

View Options

src/aphront/storage/exception/AphrontQueryConnectionException.php

View Options

src/aphront/storage/exception/AphrontQueryConnectionLostException.php

View Options

src/aphront/storage/exception/AphrontQueryCountException.php

View Options

src/aphront/storage/exception/AphrontQueryDeadlockException.php

View Options

src/aphront/storage/exception/AphrontQueryDuplicateKeyException.php

View Options

src/aphront/storage/exception/AphrontQueryException.php

View Options

src/aphront/storage/exception/AphrontQueryNotSupportedException.php

View Options

src/aphront/storage/exception/AphrontQueryObjectMissingException.php

View Options

src/aphront/storage/exception/AphrontQueryParameterException.php

View Options

src/aphront/storage/exception/AphrontQueryRecoverableException.php

View Options

src/aphront/storage/exception/AphrontQuerySchemaException.php

View Options

src/aphront/writeguard/AphrontScopedUnguardedWriteCapability.php

View Options

src/aphront/writeguard/AphrontWriteGuard.php

View Options

src/aphront/writeguard/event/AphrontWriteGuardExitEventListener.php

View Options

src/cache/PhutilKeyValueCache.php

View Options

src/cache/PhutilKeyValueCacheAPC.php

View Options

src/cache/PhutilKeyValueCacheDirectory.php

View Options

src/cache/PhutilKeyValueCacheInRequest.php

View Options

src/cache/PhutilKeyValueCacheMemcache.php

View Options

src/cache/PhutilKeyValueCacheOnDisk.php

View Options

src/cache/PhutilKeyValueCacheStack.php

View Options

src/channel/PhutilChannel.php

View Options

src/channel/PhutilChannelChannel.php

View Options

src/channel/PhutilExecChannel.php

View Options

src/channel/PhutilMetricsChannel.php

View Options

src/channel/PhutilProtocolChannel.php

View Options

src/channel/PhutilSocketChannel.php

View Options

src/conduit/ConduitClient.php

View Options

src/conduit/ConduitClientException.php

View Options

src/conduit/ConduitFuture.php

View Options

src/console/PhutilConsole.php

View Options

src/console/PhutilConsoleFormatter.php

View Options

src/console/PhutilConsoleServer.php

View Options

src/console/PhutilConsoleServerChannel.php

View Options

src/console/PhutilConsoleStdinNotInteractiveException.php

View Options

src/console/PhutilInteractiveEditor.php

View Options

src/console/__tests__/PhutilConsoleWrapTestCase.php

View Options

src/console/format.php

View Options

src/daemon/PhutilDaemon.php

View Options

src/daemon/PhutilDaemonOverseer.php

View Options

src/daemon/torture/PhutilExcessiveServiceCallsDaemon.php

View Options

src/daemon/torture/PhutilFatalDaemon.php

View Options

src/daemon/torture/PhutilHangForeverDaemon.php

View Options

src/daemon/torture/PhutilNiceDaemon.php

View Options

src/daemon/torture/PhutilProcessGroupDaemon.php

View Options

src/daemon/torture/PhutilSaturateStdoutDaemon.php

View Options

src/daemon/torture/PhutilTortureTestDaemon.php

View Options

src/error/PhutilAggregateException.php

View Options

src/error/PhutilErrorHandler.php

View Options

src/error/PhutilProxyException.php

View Options

src/error/phlog.php

View Options

src/events/PhutilEvent.php

View Options

src/events/PhutilEventEngine.php

View Options

src/events/PhutilEventListener.php

View Options

src/events/constant/PhutilEventConstants.php

View Options

src/events/constant/PhutilEventType.php

View Options

src/filesystem/FileFinder.php

View Options

src/filesystem/FileList.php

View Options

src/filesystem/Filesystem.php

View Options

src/filesystem/FilesystemException.php

View Options

src/filesystem/PhutilDeferredLog.php

View Options

src/filesystem/PhutilDirectoryFixture.php

View Options

src/filesystem/PhutilFileLock.php

View Options

src/filesystem/PhutilLock.php

View Options

src/filesystem/TempFile.php

View Options

src/filesystem/__tests__/FileFinderTestCase.php

View Options

src/filesystem/__tests__/FilesystemTestCase.php

View Options

src/filesystem/__tests__/PhutilDeferredLogTestCase.php

View Options

src/filesystem/__tests__/PhutilFileLockTestCase.php

View Options

src/filesystem/linesofalarge/LinesOfALarge.php

View Options

src/filesystem/linesofalarge/LinesOfALargeExecFuture.php

View Options

src/filesystem/linesofalarge/LinesOfALargeFile.php

View Options

src/filesystem/linesofalarge/__tests__/LinesOfALargeExecFutureTestCase.php

View Options

src/filesystem/linesofalarge/__tests__/LinesOfALargeFileTestCase.php

View Options

src/future/Future.php

View Options

src/future/FutureIterator.php

View Options

src/future/FutureProxy.php

View Options

src/future/ImmediateFuture.php

View Options

src/future/__tests__/FutureIteratorTestCase.php

View Options

src/future/asana/PhutilAsanaFuture.php

View Options

src/future/aws/PhutilAWSEC2Future.php

View Options

src/future/aws/PhutilAWSException.php

View Options

src/future/aws/PhutilAWSFuture.php

View Options

src/future/aws/PhutilAWSS3Future.php

View Options

src/future/exec/CommandException.php

View Options

src/future/exec/ExecFuture.php

View Options

src/future/exec/PhutilExecPassthru.php

View Options

src/future/exec/__tests__/ExecFutureTestCase.php

View Options

src/future/exec/execx.php

View Options

src/future/functions.php

View Options

src/future/http/BaseHTTPFuture.php

View Options

src/future/http/HTTPFuture.php

View Options

src/future/http/HTTPSFuture.php

View Options

src/future/http/status/HTTPFutureResponseStatus.php

View Options

src/future/http/status/HTTPFutureResponseStatusCURL.php

View Options

src/future/http/status/HTTPFutureResponseStatusHTTP.php

View Options

src/future/http/status/HTTPFutureResponseStatusParse.php

View Options

src/future/http/status/HTTPFutureResponseStatusTransport.php

View Options

src/future/paypal/PhutilPayPalAPIFuture.php

View Options

src/future/twitch/PhutilTwitchFuture.php

View Options

src/internationalization/PhutilPerson.php

View Options

src/internationalization/PhutilTranslator.php

View Options

src/internationalization/__tests__/PhutilPHTTestCase.php

View Options

src/internationalization/__tests__/PhutilPersonTest.php

View Options

src/internationalization/__tests__/PhutilTranslatorTestCase.php

View Options

src/internationalization/pht.php

View Options

src/lexer/PhutilLexer.php

View Options

src/lexer/PhutilPHPFragmentLexer.php

View Options

src/lexer/PhutilPythonFragmentLexer.php

View Options

src/lexer/PhutilShellLexer.php

View Options

src/lexer/PhutilSimpleOptionsLexer.php

View Options

src/lexer/PhutilTypeLexer.php

View Options

src/markup/PhutilMarkupEngine.php

View Options

src/markup/__tests__/PhutilMarkupTestCase.php

View Options

src/markup/__tests__/PhutilSafeHTMLTestCase.php

View Options

src/markup/engine/__tests__/PhutilRemarkupEngineTestCase.php

View Options

src/markup/engine/remarkup/PhutilRemarkupBlockStorage.php

View Options

src/markup/engine/remarkup/blockrule/PhutilRemarkupBlockInterpreter.php

View Options

src/markup/engine/remarkup/blockrule/PhutilRemarkupEngineBlockRule.php

View Options

src/markup/engine/remarkup/blockrule/PhutilRemarkupEngineRemarkupCodeBlockRule.php

View Options

src/markup/engine/remarkup/blockrule/PhutilRemarkupEngineRemarkupDefaultBlockRule.php

View Options

src/markup/engine/remarkup/blockrule/PhutilRemarkupEngineRemarkupHeaderBlockRule.php

View Options

src/markup/engine/remarkup/blockrule/PhutilRemarkupEngineRemarkupHorizontalRuleBlockRule.php

View Options

src/markup/engine/remarkup/blockrule/PhutilRemarkupEngineRemarkupInlineBlockRule.php

View Options

src/markup/engine/remarkup/blockrule/PhutilRemarkupEngineRemarkupInterpreterRule.php

View Options

src/markup/engine/remarkup/blockrule/PhutilRemarkupEngineRemarkupListBlockRule.php

View Options

src/markup/engine/remarkup/blockrule/PhutilRemarkupEngineRemarkupLiteralBlockRule.php

View Options

src/markup/engine/remarkup/blockrule/PhutilRemarkupEngineRemarkupNoteBlockRule.php

View Options

src/markup/engine/remarkup/blockrule/PhutilRemarkupEngineRemarkupSimpleTableBlockRule.php

View Options

src/markup/engine/remarkup/blockrule/PhutilRemarkupEngineRemarkupTableBlockRule.php

View Options

src/markup/engine/remarkup/blockrule/PhutilRemarkupEngineRemarkupTestInterpreterRule.php

View Options

src/markup/engine/remarkup/markuprule/PhutilRemarkupRule.php

View Options

src/markup/engine/remarkup/markuprule/PhutilRemarkupRuleBold.php

View Options

src/markup/engine/remarkup/markuprule/PhutilRemarkupRuleDel.php

View Options

src/markup/engine/remarkup/markuprule/PhutilRemarkupRuleDocumentLink.php

View Options

src/markup/engine/remarkup/markuprule/PhutilRemarkupRuleEscapeRemarkup.php

View Options

src/markup/engine/remarkup/markuprule/PhutilRemarkupRuleHyperlink.php

View Options

src/markup/engine/remarkup/markuprule/PhutilRemarkupRuleItalic.php

View Options

src/markup/engine/remarkup/markuprule/PhutilRemarkupRuleLinebreaks.php

View Options

src/markup/engine/remarkup/markuprule/PhutilRemarkupRuleMonospace.php

View Options

src/markup/render.php

View Options

src/markup/syntax/engine/PhutilDefaultSyntaxHighlighterEngine.php

View Options

src/markup/syntax/engine/PhutilSyntaxHighlighterEngine.php

View Options

src/markup/syntax/engine/__tests__/PhutilDefaultSyntaxHighlighterEngineTestCase.php

View Options

src/markup/syntax/highlighter/PhutilConsoleSyntaxHighlighter.php

View Options

src/markup/syntax/highlighter/PhutilDefaultSyntaxHighlighter.php

View Options

src/markup/syntax/highlighter/PhutilDivinerSyntaxHighlighter.php

View Options

src/markup/syntax/highlighter/PhutilInvisibleSyntaxHighlighter.php

View Options

src/markup/syntax/highlighter/PhutilLexerSyntaxHighlighter.php

View Options

src/markup/syntax/highlighter/PhutilPygmentsSyntaxHighlighter.php

View Options

src/markup/syntax/highlighter/PhutilRainbowSyntaxHighlighter.php

View Options

src/markup/syntax/highlighter/PhutilSyntaxHighlighter.php

View Options

src/markup/syntax/highlighter/PhutilSyntaxHighlighterException.php

View Options

src/markup/syntax/highlighter/PhutilXHPASTSyntaxHighlighter.php

View Options

src/markup/syntax/highlighter/__tests__/PhutilPHPFragmentLexerHighlighterTestCase.php

View Options

src/markup/syntax/highlighter/__tests__/PhutilXHPASTSyntaxHighlighterTestCase.php

View Options

src/markup/syntax/highlighter/pygments/PhutilDefaultSyntaxHighlighterEnginePygmentsFuture.php

View Options

src/markup/syntax/highlighter/xhpast/PhutilXHPASTSyntaxHighlighterFuture.php

View Options

src/moduleutils/PhutilBootloader.php

View Options

src/moduleutils/PhutilBootloaderException.php

View Options

src/moduleutils/PhutilLibraryConflictException.php

View Options

src/moduleutils/core.php

View Options

src/parser/PhutilDocblockParser.php

View Options

src/parser/__tests__/PhutilDocblockParserTestCase.php

View Options

src/parser/__tests__/PhutilEmailAddressTestCase.php

View Options

src/parser/__tests__/PhutilJSONTestCase.php

View Options

src/parser/__tests__/PhutilLanguageGuesserTestCase.php

View Options

src/parser/__tests__/PhutilQueryStringParserTestCase.php

View Options

src/parser/__tests__/PhutilSimpleOptionsTestCase.php

View Options

src/parser/__tests__/PhutilURITestCase.php

View Options

src/parser/aast/api/AASTNode.php

View Options

src/parser/aast/api/AASTNodeList.php

View Options

src/parser/aast/api/AASTToken.php

View Options

src/parser/aast/api/AASTTree.php

View Options

src/parser/argument/PhutilArgumentParser.php

View Options

src/parser/argument/PhutilArgumentSpecification.php

View Options

src/parser/argument/__tests__/PhutilArgumentParserTestCase.php

View Options

src/parser/argument/__tests__/PhutilArgumentSpecificationTestCase.php

View Options

src/parser/argument/exception/PhutilArgumentParserException.php

View Options

src/parser/argument/exception/PhutilArgumentSpecificationException.php

View Options

src/parser/argument/exception/PhutilArgumentUsageException.php

View Options

src/parser/argument/workflow/PhutilArgumentWorkflow.php

View Options

src/parser/argument/workflow/PhutilHelpArgumentWorkflow.php

View Options

src/parser/xhpast/api/XHPASTNode.php

View Options

src/parser/xhpast/api/XHPASTSyntaxErrorException.php

View Options

src/parser/xhpast/api/XHPASTToken.php

View Options

src/parser/xhpast/api/XHPASTTree.php

View Options

src/parser/xhpast/api/__tests__/XHPASTTreeTestCase.php

View Options

src/readableserializer/PhutilReadableSerializer.php

View Options

src/symbols/PhutilSymbolLoader.php

View Options

src/symbols/exception/PhutilMissingSymbolException.php

View Options

src/utils/PhutilUTF8StringTruncator.php

View Options

src/utils/__tests__/AbstractDirectedGraphTestCase.php

View Options

src/utils/__tests__/MFilterTestHelper.php

View Options

src/utils/__tests__/PhutilArrayTestCase.php

View Options

src/utils/__tests__/PhutilBufferedIteratorTestCase.php

View Options

src/utils/__tests__/PhutilChunkedIteratorTestCase.php

View Options

src/utils/__tests__/PhutilUTF8TestCase.php

View Options

src/utils/__tests__/PhutilUtilsTestCase.php

View Options

src/utils/__tests__/TestAbstractDirectedGraph.php

View Options

src/utils/utf8.php

View Options

src/utils/utils.php

View Options

src/xsprintf/PhutilQsprintfInterface.php

View Options

src/xsprintf/csprintf.php

View Options

src/xsprintf/hgsprintf.php

View Options

src/xsprintf/jsprintf.php

View Options

src/xsprintf/ldapsprintf.php

View Options

src/xsprintf/qsprintf.php

View Options

src/xsprintf/queryfx.php

View Options

src/xsprintf/urisprintf.php

Phabricator · Built by Phacility