@@ -43,8 +43,8 @@ |
||
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 |
@@ -55,7 +55,7 @@ |
||
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 | ]); |
@@ -37,17 +37,17 @@ discard block |
||
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 |
||
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 | } |
@@ -26,8 +26,8 @@ discard block |
||
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 |
||
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 |
@@ -105,7 +105,7 @@ discard block |
||
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 |
||
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 | /** |
@@ -119,9 +119,9 @@ discard block |
||
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 |
||
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 |
||
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 |
||
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 | /** |
@@ -41,55 +41,55 @@ discard block |
||
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 |
||
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 | } |
@@ -175,7 +175,7 @@ |
||
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 |
@@ -103,7 +103,7 @@ |
||
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 | } |