Remove @group annotations

Summary: I'm pretty sure that `@group` annotations are useless now... see D9855. Also fixed various other minor issues.

Test Plan: Eye-ball it.

Reviewers: #blessed_reviewers, epriestley, chad

Reviewed By: #blessed_reviewers, epriestley

Subscribers: epriestley, Korvin, hach-que

Differential Revision: https://secure.phabricator.com/D9859

Author: joshuaspence

conf/default.conf.php

@@ -798,7 +798,7 @@
     // '#/autobuilt/#',
   ),
 
-  // If you set this to true, users can accept their own revisions.  This action
+  // If you set this to true, users can accept their own revisions. This action
   // is disabled by default because it's most likely not a behavior you want,
   // but it proves useful if you are working alone on a project and want to make
   // use of all of differential's features.
@@ -817,7 +817,7 @@
 
   // If you set this to true, any user can reopen a revision so long as it has
   // been closed.  This can be useful if a revision is accidentally closed or
-  // if a developer changes his or her mind after closing a revision.  If it is
+  // if a developer changes his or her mind after closing a revision. If it is
   // false, reopening is not allowed.
   'differential.allow-reopen' => false,
 

externals/balanced-php/tests/Balanced/SuiteTest.php

@@ -27,8 +27,6 @@
  *     <li>$BALANCED_URL_ROOT If set applies to \Balanced\Settings::$url_root.
  *     <li>$BALANCED_API_KEY If set applies to \Balanced\Settings::$api_key.
  * </ul>
- *
- * @group suite
  */
 class SuiteTest extends \PHPUnit_Framework_TestCase
 {
@@ -784,17 +782,17 @@ function testBankAccountVerificationSuccess() {
     }
 
     function testEvents() {
-        $prev_num_events = Marketplace::mine()->events->total();
-        $account = self::_createBuyer();
-        $account->debit(123);
-        $cur_num_events = Marketplace::mine()->events->total();
-        $count = 0;
+        $prev_num_events = Marketplace::mine()->events->total();
+        $account = self::_createBuyer();
+        $account->debit(123);
+        $cur_num_events = Marketplace::mine()->events->total();
+        $count = 0;
         while ($cur_num_events == $prev_num_events && $count < 10) {
             printf("waiting for events - %d, %d == %d\n", $count + 1, $cur_num_events, $prev_num_events);
             sleep(2); // 2 seconds
-            $cur_num_events = Marketplace::mine()->events->total();
-            $count += 1;
-        }
+            $cur_num_events = Marketplace::mine()->events->total();
+            $count += 1;
+        }
         $this->assertTrue($cur_num_events > $prev_num_events);
     }
 }

src/aphront/AphrontController.php

@@ -1,8 +1,5 @@
 <?php
 
-/**
- * @group aphront
- */
 abstract class AphrontController extends Phobject {
 
   private $request;

src/aphront/AphrontURIMapper.php

@@ -1,8 +1,5 @@
 <?php
 
-/**
- * @group aphront
- */
 final class AphrontURIMapper {
 
   private $map;
@@ -49,4 +46,5 @@ final private function tryRule($rule, $value, $path) {
 
     return array(null, null);
   }
+
 }

src/aphront/configuration/AphrontApplicationConfiguration.php

@@ -2,7 +2,6 @@
 
 /**
  * @task  routing URI Routing
- * @group aphront
  */
 abstract class AphrontApplicationConfiguration {
 
@@ -228,4 +227,5 @@ final public function buildControllerForPath($path) {
 
     return array($controller, $uri_data);
   }
+
 }

src/aphront/configuration/AphrontDefaultApplicationConfiguration.php

@@ -4,7 +4,6 @@
  * NOTE: Do not extend this!
  *
  * @concrete-extensible
- * @group aphront
  */
 class AphrontDefaultApplicationConfiguration
   extends AphrontApplicationConfiguration {

src/aphront/console/DarkConsoleController.php

@@ -1,8 +1,5 @@
 <?php
 
-/**
- * @group console
- */
 final class DarkConsoleController extends PhabricatorController {
 
   protected $op;

src/aphront/console/DarkConsoleCore.php

@@ -1,8 +1,5 @@
 <?php
 
-/**
- * @group console
- */
 final class DarkConsoleCore {
 
   private $plugins = array();

src/aphront/console/DarkConsoleDataController.php

@@ -1,8 +1,5 @@
 <?php
 
-/**
- * @group console
- */
 final class DarkConsoleDataController extends PhabricatorController {
 
   private $key;

src/aphront/console/plugin/DarkConsoleErrorLogPlugin.php

@@ -1,8 +1,5 @@
 <?php
 
-/**
- * @group console
- */
 final class DarkConsoleErrorLogPlugin extends DarkConsolePlugin {
 
   public function getName() {
@@ -98,4 +95,5 @@ public function renderPanel() {
         phutil_tag('pre', array('class' => 'PhabricatorMonospaced'), $details),
       ));
   }
+
 }