Passed
Pull Request — master (#103)
by Sebastian
02:29
created
src/Hook/Message/Action/Beams.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -43,8 +43,8 @@
 block discarded – undo
43 43
         $options = $action->getOptions();
44 44
         $book    = new RuleBook();
45 45
         $book->setRules(RuleBook\RuleSet::beams(
46
-            (int)  $options->get('subjectLength', 50),
47
-            (int)  $options->get('bodyLineLength', 72),
46
+            (int) $options->get('subjectLength', 50),
47
+            (int) $options->get('bodyLineLength', 72),
48 48
             (bool) $options->get('checkImperativeBeginningOnly', false)
49 49
         ));
50 50
 
Please login to merge, or discard this patch.
tests/CaptainHook/Console/Command/EnableTest.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -55,7 +55,7 @@
 block discarded – undo
55 55
             null,
56 56
             ['captainhook.json' => file_get_contents(CH_PATH_FILES . '/config/valid.json')]
57 57
         );
58
-        $input      = new ArrayInput([
58
+        $input = new ArrayInput([
59 59
             'hook'            => 'pre-push',
60 60
             '--configuration' => $fakeConfig->url() . '/captainhook.json',
61 61
         ]);
Please login to merge, or discard this patch.
src/Console/Command/Configuration.php 1 patch
Indentation   +15 added lines, -15 removed lines patch added patch discarded remove patch
@@ -37,17 +37,17 @@  discard block
 block discarded – undo
37 37
     {
38 38
         parent::configure();
39 39
         $this->setName('configure')
40
-             ->setDescription('Configure your hooks')
41
-             ->setHelp('This command creates or updates your captainhook configuration')
42
-             ->addOption('extend', 'e', InputOption::VALUE_NONE, 'Extend existing configuration file')
43
-             ->addOption('force', 'f', InputOption::VALUE_NONE, 'Overwrite existing configuration file')
44
-             ->addOption('advanced', 'a', InputOption::VALUE_NONE, 'More options, but more to type')
45
-             ->addOption(
46
-                 'bootstrap',
47
-                 null,
48
-                 InputOption::VALUE_OPTIONAL,
49
-                 'Path to composers vendor/autoload.php'
50
-             );
40
+                ->setDescription('Configure your hooks')
41
+                ->setHelp('This command creates or updates your captainhook configuration')
42
+                ->addOption('extend', 'e', InputOption::VALUE_NONE, 'Extend existing configuration file')
43
+                ->addOption('force', 'f', InputOption::VALUE_NONE, 'Overwrite existing configuration file')
44
+                ->addOption('advanced', 'a', InputOption::VALUE_NONE, 'More options, but more to type')
45
+                ->addOption(
46
+                    'bootstrap',
47
+                    null,
48
+                    InputOption::VALUE_OPTIONAL,
49
+                    'Path to composers vendor/autoload.php'
50
+                );
51 51
     }
52 52
 
53 53
     /**
@@ -65,10 +65,10 @@  discard block
 block discarded – undo
65 65
 
66 66
         $configurator = new Creator($io, $config);
67 67
         $configurator->force(IOUtil::argToBool($input->getOption('force')))
68
-                     ->extend(IOUtil::argToBool($input->getOption('extend')))
69
-                     ->advanced(IOUtil::argToBool($input->getOption('advanced')))
70
-                     ->setExecutable($this->resolver->getExecutable())
71
-                     ->run();
68
+                        ->extend(IOUtil::argToBool($input->getOption('extend')))
69
+                        ->advanced(IOUtil::argToBool($input->getOption('advanced')))
70
+                        ->setExecutable($this->resolver->getExecutable())
71
+                        ->run();
72 72
         return 0;
73 73
     }
74 74
 }
Please login to merge, or discard this patch.
tests/CaptainHook/Console/ApplicationTest.php 1 patch
Indentation   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -26,8 +26,8 @@  discard block
 block discarded – undo
26 26
     {
27 27
         $input = new ArrayInput(['--version' => true]);
28 28
         $output = $this->getMockBuilder(NullOutput::class)
29
-                       ->disableOriginalConstructor()
30
-                       ->getMock();
29
+                        ->disableOriginalConstructor()
30
+                        ->getMock();
31 31
 
32 32
         $output->expects($this->once())->method('writeLn');
33 33
 
@@ -46,8 +46,8 @@  discard block
 block discarded – undo
46 46
     {
47 47
         $input = new ArrayInput(['command' => 'list']);
48 48
         $output = $this->getMockBuilder(NullOutput::class)
49
-                       ->disableOriginalConstructor()
50
-                       ->getMock();
49
+                        ->disableOriginalConstructor()
50
+                        ->getMock();
51 51
 
52 52
         $output->expects($this->atLeastOnce())->method('write');
53 53
 
Please login to merge, or discard this patch.
src/Runner/Action/PHP.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -105,7 +105,7 @@  discard block
 block discarded – undo
105 105
         }
106 106
         ob_start();
107 107
         $class::$method();
108
-        return (string)ob_get_clean();
108
+        return (string) ob_get_clean();
109 109
     }
110 110
 
111 111
     /**
@@ -134,7 +134,7 @@  discard block
 block discarded – undo
134 134
      */
135 135
     private function isStaticMethodCall(string $class): bool
136 136
     {
137
-        return (bool)preg_match('#^\\\\.+::.+$#i', $class);
137
+        return (bool) preg_match('#^\\\\.+::.+$#i', $class);
138 138
     }
139 139
 
140 140
     /**
Please login to merge, or discard this patch.
tests/CaptainHook/Runner/InstallerTest.php 1 patch
Indentation   +15 added lines, -15 removed lines patch added patch discarded remove patch
@@ -119,9 +119,9 @@  discard block
 block discarded – undo
119 119
         $template = $this->createTemplateMock();
120 120
 
121 121
         $template->expects($this->once())
122
-                 ->method('getCode')
123
-                 ->with('pre-commit')
124
-                 ->willReturn('');
122
+                    ->method('getCode')
123
+                    ->with('pre-commit')
124
+                    ->willReturn('');
125 125
 
126 126
         $runner = new Installer($io, $config, $repo, $template);
127 127
         $runner->setHook('pre-commit');
@@ -156,14 +156,14 @@  discard block
 block discarded – undo
156 156
         $repo     = new Repository($fakeRepo->getRoot());
157 157
 
158 158
         $template->expects($this->once())
159
-                 ->method('getCode')
160
-                 ->with('pre-commit')
161
-                 ->willReturn('');
159
+                    ->method('getCode')
160
+                    ->with('pre-commit')
161
+                    ->willReturn('');
162 162
 
163 163
         $runner = new Installer($io, $config, $repo, $template);
164 164
         $runner->setHook('pre-commit')
165
-               ->setMoveExistingTo('foo/bar/')
166
-               ->run();
165
+                ->setMoveExistingTo('foo/bar/')
166
+                ->run();
167 167
 
168 168
         $this->assertFileExists($fakeRepo->getHookDir() . '/pre-commit');
169 169
         $this->assertFileExists($fakeRepo->getRoot() . '/foo/bar/pre-commit');
@@ -194,14 +194,14 @@  discard block
 block discarded – undo
194 194
         $repo     = new Repository($fakeRepo->getRoot());
195 195
 
196 196
         $template->expects($this->once())
197
-                 ->method('getCode')
198
-                 ->with('pre-commit')
199
-                 ->willReturn('');
197
+                    ->method('getCode')
198
+                    ->with('pre-commit')
199
+                    ->willReturn('');
200 200
 
201 201
         $runner = new Installer($io, $config, $repo, $template);
202 202
         $runner->setHook('pre-commit')
203
-               ->setMoveExistingTo('foo/bar/')
204
-               ->run();
203
+                ->setMoveExistingTo('foo/bar/')
204
+                ->run();
205 205
 
206 206
         $this->assertFileExists($fakeRepo->getHookDir() . '/pre-commit');
207 207
     }
@@ -235,8 +235,8 @@  discard block
 block discarded – undo
235 235
 
236 236
         $runner = new Installer($io, $config, $repo, $template);
237 237
         $runner->setHook('pre-commit')
238
-               ->setMoveExistingTo('foo')
239
-               ->run();
238
+                ->setMoveExistingTo('foo')
239
+                ->run();
240 240
     }
241 241
 
242 242
     /**
Please login to merge, or discard this patch.
src/Console/Command/Install.php 1 patch
Indentation   +53 added lines, -53 removed lines patch added patch discarded remove patch
@@ -41,55 +41,55 @@  discard block
 block discarded – undo
41 41
     {
42 42
         parent::configure();
43 43
         $this->setName('install')
44
-             ->setDescription('Install git hooks')
45
-             ->setHelp('This command will install the git hooks to your .git directory')
46
-             ->addArgument(
47
-                 'hook',
48
-                 InputArgument::OPTIONAL,
49
-                 'Limit the hook you want to install. By default all hooks get installed.'
50
-             )
51
-             ->addOption(
52
-                 'force',
53
-                 'f',
54
-                 InputOption::VALUE_NONE,
55
-                 'Force install without confirmation'
56
-             )
57
-             ->addOption(
58
-                 'skip-existing',
59
-                 's',
60
-                 InputOption::VALUE_NONE,
61
-                 'Do not overwrite existing hooks'
62
-             )
63
-             ->addOption(
64
-                 'move-existing-to',
65
-                 null,
66
-                 InputOption::VALUE_OPTIONAL,
67
-                 'Move existing hooks to given directory'
68
-             )
69
-             ->addOption(
70
-                 'bootstrap',
71
-                 'b',
72
-                 InputOption::VALUE_OPTIONAL,
73
-                 'Path to composers vendor/autoload.php'
74
-             )
75
-             ->addOption(
76
-                 'run-mode',
77
-                 'm',
78
-                 InputOption::VALUE_OPTIONAL,
79
-                 'Git hook run mode [php|shell|docker]'
80
-             )
81
-             ->addOption(
82
-                 'run-exec',
83
-                 'e',
84
-                 InputOption::VALUE_OPTIONAL,
85
-                 'The Docker command to start your container e.g. \'docker exec CONTAINER\''
86
-             )
87
-             ->addOption(
88
-                 'run-path',
89
-                 'p',
90
-                 InputOption::VALUE_OPTIONAL,
91
-                 'The path to the CaptainHook executable \'/usr/bin/captainhook\''
92
-             );
44
+                ->setDescription('Install git hooks')
45
+                ->setHelp('This command will install the git hooks to your .git directory')
46
+                ->addArgument(
47
+                    'hook',
48
+                    InputArgument::OPTIONAL,
49
+                    'Limit the hook you want to install. By default all hooks get installed.'
50
+                )
51
+                ->addOption(
52
+                    'force',
53
+                    'f',
54
+                    InputOption::VALUE_NONE,
55
+                    'Force install without confirmation'
56
+                )
57
+                ->addOption(
58
+                    'skip-existing',
59
+                    's',
60
+                    InputOption::VALUE_NONE,
61
+                    'Do not overwrite existing hooks'
62
+                )
63
+                ->addOption(
64
+                    'move-existing-to',
65
+                    null,
66
+                    InputOption::VALUE_OPTIONAL,
67
+                    'Move existing hooks to given directory'
68
+                )
69
+                ->addOption(
70
+                    'bootstrap',
71
+                    'b',
72
+                    InputOption::VALUE_OPTIONAL,
73
+                    'Path to composers vendor/autoload.php'
74
+                )
75
+                ->addOption(
76
+                    'run-mode',
77
+                    'm',
78
+                    InputOption::VALUE_OPTIONAL,
79
+                    'Git hook run mode [php|shell|docker]'
80
+                )
81
+                ->addOption(
82
+                    'run-exec',
83
+                    'e',
84
+                    InputOption::VALUE_OPTIONAL,
85
+                    'The Docker command to start your container e.g. \'docker exec CONTAINER\''
86
+                )
87
+                ->addOption(
88
+                    'run-path',
89
+                    'p',
90
+                    InputOption::VALUE_OPTIONAL,
91
+                    'The path to the CaptainHook executable \'/usr/bin/captainhook\''
92
+                );
93 93
     }
94 94
 
95 95
     /**
@@ -115,10 +115,10 @@  discard block
 block discarded – undo
115 115
         $template  = Template\Builder::build($config, $repo, $this->resolver);
116 116
         $installer = new Installer($io, $config, $repo, $template);
117 117
         $installer->setForce(IOUtil::argToBool($input->getOption('force')))
118
-                  ->setSkipExisting(IOUtil::argToBool($input->getOption('skip-existing')))
119
-                  ->setMoveExistingTo(IOUtil::argToString($input->getOption('move-existing-to')))
120
-                  ->setHook(IOUtil::argToString($input->getArgument('hook')))
121
-                  ->run();
118
+                    ->setSkipExisting(IOUtil::argToBool($input->getOption('skip-existing')))
119
+                    ->setMoveExistingTo(IOUtil::argToString($input->getOption('move-existing-to')))
120
+                    ->setHook(IOUtil::argToString($input->getArgument('hook')))
121
+                    ->run();
122 122
 
123 123
         return 0;
124 124
     }
Please login to merge, or discard this patch.
src/Runner/Installer.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -175,7 +175,7 @@
 block discarded – undo
175 175
         // callback to write bool true to all array entries
176 176
         // to make sure the user will be asked to confirm every hook installation
177 177
         // unless the user provided the force or skip option
178
-        $callback = function () {
178
+        $callback = function() {
179 179
             return true;
180 180
         };
181 181
         // if a specific hook is set the user chose it so don't ask for permission anymore
Please login to merge, or discard this patch.
src/Hook/Condition/FileChanged.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -103,7 +103,7 @@
 block discarded – undo
103 103
             $info = explode(' ', $stdIn[0]);
104 104
             // If we find a rewritten commit, we return the first commit before the rewritten one.
105 105
             // If we do not find any rewritten commits (awkward) we use the last ref-log position.
106
-            return isset($info[1]) ? $info[1] . '^' :  'HEAD@{1}';
106
+            return isset($info[1]) ? $info[1] . '^' : 'HEAD@{1}';
107 107
         }
108 108
         return $io->getArgument('previousHead', 'HEAD@{1}');
109 109
     }
Please login to merge, or discard this patch.