morphatic / laravel-auto-deploy

src/Controllers/DeployController.php

Doc Comments +3 added lines, -3 removed lines

@@ -75,8 +75,8 @@ 
     /**
      * Create a new DeployController instance.
      *
-     * @param Morphatic\AutoDeploy\Origins\OriginInterface $origin The origin of the webhook
-     * @param AdamBrett\ShellWrapper\Runners\Exec          $exec   The shell command execution class
+     * @param OriginInterface $origin The origin of the webhook
+     * @param Exec          $exec   The shell command execution class
      */
     public function __construct(OriginInterface $origin, Exec $exec)
     {
@@ -131,7 +131,7 @@ 
     /**
      * Runs a shell command, logs, and handles the result.
      *
-     * @param AdamBrett\ShellWrapper\CommandInterface $cmd The text of the command to be run
+     * @param CommandInterface $cmd The text of the command to be run
      *
      * @return bool True if the command was successful, false on error
      */

Unused Use Statements +4 added lines, -5 removed lines

@@ -2,16 +2,15 @@
 
 namespace Morphatic\AutoDeploy\Controllers;
 
-use Log;
-use Mail;
-use Monolog\Logger;
-use Illuminate\Http\Request;
-use Illuminate\Routing\Controller;
 use AdamBrett\ShellWrapper\Command\Builder as Command;
 use AdamBrett\ShellWrapper\Command\CommandInterface;
 use AdamBrett\ShellWrapper\Runners\Exec;
+use Illuminate\Routing\Controller;
+use Log;
+use Mail;
 use Monolog\Formatter\HtmlFormatter;
 use Monolog\Handler\SwiftMailerHandler;
+use Monolog\Logger;
 use Morphatic\AutoDeploy\Origins\OriginInterface;
 
 class DeployController extends Controller

Spacing +1 added lines, -1 removed lines

@@ -119,7 +119,7 @@
             'Timestamp' => date('r'),
             'output' => '',
         ];
-        $whitelist = ['backupDatabase','pull','copyEnv','composer','npm','migrate','seed','deploy'];
+        $whitelist = ['backupDatabase', 'pull', 'copyEnv', 'composer', 'npm', 'migrate', 'seed', 'deploy'];
         foreach ($steps as $step) {
             if (in_array($step, $whitelist) && !$this->{$step}()) {
                 $this->log->error('Deploy failed.', $this->result);

src/Origins/Github.php

Doc Comments +1 added lines, -1 removed lines

@@ -31,7 +31,7 @@
      *
      * Follows the procedure described here: https://developer.github.com/webhooks/securing/
      *
-     * @return bool Returns true if the request is authentic. False otherwise.
+     * @return null|boolean Returns true if the request is authentic. False otherwise.
      */
     public function isAuthentic()
     {

src/Commands/DeployInitCommand.php

Indentation +3 added lines, -3 removed lines

@@ -33,9 +33,9 @@
         $url = parse_url(config('app.url'), PHP_URL_HOST);
         $ssl = config('auto-deploy.require-ssl') ? 'https' : 'http';
         $msg = "Here is the information you'll need to set up your webhooks:\n\n".
-               "  Payload URL: <comment>$ssl://$url/%s</comment>\n".
-               "  Secret Key:  <comment>%s</comment>\n\n".
-               "You can display this information again by running `php artisan deploy:info`\n";
+                "  Payload URL: <comment>$ssl://$url/%s</comment>\n".
+                "  Secret Key:  <comment>%s</comment>\n\n".
+                "You can display this information again by running `php artisan deploy:info`\n";
         $conf = 'Are you sure you want to overwrite the existing keys?';
         $show = $this->option('show');
         $over = $this->option('force');