line |
stmt |
bran |
cond |
sub |
pod |
time |
code |
1
|
|
|
|
|
|
|
package App::GitHooks::Hook::PreCommit; |
2
|
|
|
|
|
|
|
|
3
|
1
|
|
|
1
|
|
18352
|
use strict; |
|
1
|
|
|
|
|
2
|
|
|
1
|
|
|
|
|
34
|
|
4
|
1
|
|
|
1
|
|
3
|
use warnings; |
|
1
|
|
|
|
|
2
|
|
|
1
|
|
|
|
|
24
|
|
5
|
|
|
|
|
|
|
|
6
|
|
|
|
|
|
|
# Inherit from the base Hook class. |
7
|
1
|
|
|
1
|
|
3
|
use base 'App::GitHooks::Hook'; |
|
1
|
|
|
|
|
2
|
|
|
1
|
|
|
|
|
351
|
|
8
|
|
|
|
|
|
|
|
9
|
|
|
|
|
|
|
# External dependencies. |
10
|
1
|
|
|
1
|
|
7
|
use Carp; |
|
1
|
|
|
|
|
1
|
|
|
1
|
|
|
|
|
44
|
|
11
|
1
|
|
|
1
|
|
645
|
use Data::Dumper; |
|
1
|
|
|
|
|
7222
|
|
|
1
|
|
|
|
|
62
|
|
12
|
1
|
|
|
1
|
|
491
|
use Data::Validate::Type; |
|
1
|
|
|
|
|
6227
|
|
|
1
|
|
|
|
|
50
|
|
13
|
1
|
|
|
1
|
|
795
|
use Path::Tiny qw(); |
|
1
|
|
|
|
|
11001
|
|
|
1
|
|
|
|
|
33
|
|
14
|
|
|
|
|
|
|
|
15
|
|
|
|
|
|
|
# Internal dependencies. |
16
|
1
|
|
|
1
|
|
6
|
use App::GitHooks::Constants qw( :HOOK_EXIT_CODES :PLUGIN_RETURN_CODES ); |
|
1
|
|
|
|
|
2
|
|
|
1
|
|
|
|
|
470
|
|
17
|
|
|
|
|
|
|
|
18
|
|
|
|
|
|
|
|
19
|
|
|
|
|
|
|
=head1 NAME |
20
|
|
|
|
|
|
|
|
21
|
|
|
|
|
|
|
App::GitHooks::Hook::PreCommit - Handler for pre-commit hook. |
22
|
|
|
|
|
|
|
|
23
|
|
|
|
|
|
|
|
24
|
|
|
|
|
|
|
=head1 VERSION |
25
|
|
|
|
|
|
|
|
26
|
|
|
|
|
|
|
Version 1.7.3 |
27
|
|
|
|
|
|
|
|
28
|
|
|
|
|
|
|
=cut |
29
|
|
|
|
|
|
|
|
30
|
|
|
|
|
|
|
our $VERSION = '1.7.3'; |
31
|
|
|
|
|
|
|
|
32
|
|
|
|
|
|
|
|
33
|
|
|
|
|
|
|
=head1 METHODS |
34
|
|
|
|
|
|
|
|
35
|
|
|
|
|
|
|
=head2 run() |
36
|
|
|
|
|
|
|
|
37
|
|
|
|
|
|
|
Run the hook handler and return an exit status to pass to git. |
38
|
|
|
|
|
|
|
|
39
|
|
|
|
|
|
|
my $exit_status = App::GitHooks::Hook::PreCommit->run( |
40
|
|
|
|
|
|
|
app => $app, |
41
|
|
|
|
|
|
|
); |
42
|
|
|
|
|
|
|
|
43
|
|
|
|
|
|
|
Arguments: |
44
|
|
|
|
|
|
|
|
45
|
|
|
|
|
|
|
=over 4 |
46
|
|
|
|
|
|
|
|
47
|
|
|
|
|
|
|
=item * app I<(mandatory)> |
48
|
|
|
|
|
|
|
|
49
|
|
|
|
|
|
|
An App::GitHooks object. |
50
|
|
|
|
|
|
|
|
51
|
|
|
|
|
|
|
=back |
52
|
|
|
|
|
|
|
|
53
|
|
|
|
|
|
|
=cut |
54
|
|
|
|
|
|
|
|
55
|
|
|
|
|
|
|
sub run |
56
|
|
|
|
|
|
|
{ |
57
|
0
|
|
|
0
|
1
|
|
my ( $class, %args ) = @_; |
58
|
0
|
|
|
|
|
|
my $app = delete( $args{'app'} ); |
59
|
0
|
0
|
|
|
|
|
croak 'Unknown argument(s): ' . join( ', ', keys %args ) |
60
|
|
|
|
|
|
|
if scalar( keys %args ) != 0; |
61
|
|
|
|
|
|
|
|
62
|
|
|
|
|
|
|
# Check parameters. |
63
|
0
|
0
|
|
|
|
|
croak "The 'app' argument is mandatory" |
64
|
|
|
|
|
|
|
if !Data::Validate::Type::is_instance( $app, class => 'App::GitHooks' ); |
65
|
|
|
|
|
|
|
|
66
|
|
|
|
|
|
|
# Remove the file that we use to indicate that pre-commit checks have been |
67
|
|
|
|
|
|
|
# run for the set of staged files, in case it exists. |
68
|
0
|
|
|
|
|
|
unlink( '.git/COMMIT-MSG-CHECKS' ); |
69
|
|
|
|
|
|
|
|
70
|
|
|
|
|
|
|
# If the terminal isn't interactive, we won't have a human available to fix the |
71
|
|
|
|
|
|
|
# problems so we just let the commit go as is. |
72
|
0
|
|
|
|
|
|
my $config = $app->get_config(); |
73
|
0
|
|
|
|
|
|
my $force_interactive = $config->get( 'testing', 'force_interactive' ); |
74
|
0
|
0
|
0
|
|
|
|
return $HOOK_EXIT_SUCCESS |
75
|
|
|
|
|
|
|
if !$app->get_terminal()->is_interactive() && !$force_interactive; |
76
|
|
|
|
|
|
|
|
77
|
|
|
|
|
|
|
# Run the checks on the staged changes. |
78
|
0
|
|
|
|
|
|
my $checks_pass = run_all_tests( $app ); |
79
|
|
|
|
|
|
|
|
80
|
|
|
|
|
|
|
# If the checks passed, write a file for the prepare-commit-msg hook to know |
81
|
|
|
|
|
|
|
# that we've already run the checks and there's no need to do it a second time. |
82
|
|
|
|
|
|
|
# This is what allows it to detect when --no-verify was used. |
83
|
0
|
0
|
|
|
|
|
if ( $checks_pass ) |
84
|
|
|
|
|
|
|
{ |
85
|
0
|
|
|
|
|
|
Path::Tiny::path( '.git', 'COMMIT-MSG-CHECKS' ) |
86
|
|
|
|
|
|
|
->spew( $checks_pass ); |
87
|
|
|
|
|
|
|
} |
88
|
|
|
|
|
|
|
|
89
|
|
|
|
|
|
|
# Indicate if we should allow continuing to the commit message or not. |
90
|
0
|
0
|
|
|
|
|
return $checks_pass |
91
|
|
|
|
|
|
|
? $HOOK_EXIT_SUCCESS |
92
|
|
|
|
|
|
|
: $HOOK_EXIT_FAILURE; |
93
|
|
|
|
|
|
|
} |
94
|
|
|
|
|
|
|
|
95
|
|
|
|
|
|
|
|
96
|
|
|
|
|
|
|
=head2 run_all_tests() |
97
|
|
|
|
|
|
|
|
98
|
|
|
|
|
|
|
Run all the tests available for the pre-commit hook and return whether issues |
99
|
|
|
|
|
|
|
were detected. |
100
|
|
|
|
|
|
|
|
101
|
|
|
|
|
|
|
my $tests_success = run_all_tests( $app ); |
102
|
|
|
|
|
|
|
|
103
|
|
|
|
|
|
|
This is a two step operation: |
104
|
|
|
|
|
|
|
|
105
|
|
|
|
|
|
|
=over 4 |
106
|
|
|
|
|
|
|
|
107
|
|
|
|
|
|
|
=item 1. We load all the plugins that support "pre-commit", and run them to |
108
|
|
|
|
|
|
|
analyze the overall pre-commit operation. |
109
|
|
|
|
|
|
|
|
110
|
|
|
|
|
|
|
=item 2. Each staged file is loaded and we run plugins that support |
111
|
|
|
|
|
|
|
"pre-commit-file" on each one. |
112
|
|
|
|
|
|
|
|
113
|
|
|
|
|
|
|
=back |
114
|
|
|
|
|
|
|
|
115
|
|
|
|
|
|
|
=cut |
116
|
|
|
|
|
|
|
|
117
|
|
|
|
|
|
|
sub run_all_tests |
118
|
|
|
|
|
|
|
{ |
119
|
0
|
|
|
0
|
1
|
|
my ( $app ) = @_; |
120
|
|
|
|
|
|
|
|
121
|
|
|
|
|
|
|
# Find all the plugins that support the pre-commit hook. |
122
|
0
|
|
|
|
|
|
my $plugins = $app->get_hook_plugins( 'pre-commit' ); |
123
|
|
|
|
|
|
|
|
124
|
|
|
|
|
|
|
# Run the plugins. |
125
|
0
|
|
|
|
|
|
my $tests_success = 1; |
126
|
0
|
|
|
|
|
|
my $has_warnings = 0; |
127
|
0
|
|
|
|
|
|
foreach my $plugin ( @$plugins ) |
128
|
|
|
|
|
|
|
{ |
129
|
0
|
|
|
|
|
|
my $check_result = $plugin->run_pre_commit( |
130
|
|
|
|
|
|
|
app => $app, |
131
|
|
|
|
|
|
|
); |
132
|
|
|
|
|
|
|
|
133
|
0
|
0
|
|
|
|
|
$tests_success = 0 |
134
|
|
|
|
|
|
|
if $check_result == $PLUGIN_RETURN_FAILED; |
135
|
|
|
|
|
|
|
|
136
|
0
|
0
|
|
|
|
|
$has_warnings = 1 |
137
|
|
|
|
|
|
|
if $check_result == $PLUGIN_RETURN_WARNED; |
138
|
|
|
|
|
|
|
} |
139
|
|
|
|
|
|
|
|
140
|
|
|
|
|
|
|
# Check the changed files individually with plugins that support |
141
|
|
|
|
|
|
|
# "pre-commit-file". |
142
|
|
|
|
|
|
|
{ |
143
|
0
|
|
|
|
|
|
my ( $file_checks, $file_warnings ) = $app |
|
0
|
|
|
|
|
|
|
144
|
|
|
|
|
|
|
->get_staged_changes() |
145
|
|
|
|
|
|
|
->verify(); |
146
|
0
|
0
|
|
|
|
|
$tests_success = 0 |
147
|
|
|
|
|
|
|
if !$file_checks; |
148
|
0
|
0
|
|
|
|
|
$has_warnings = 1 |
149
|
|
|
|
|
|
|
if $file_warnings; |
150
|
|
|
|
|
|
|
} |
151
|
|
|
|
|
|
|
|
152
|
|
|
|
|
|
|
# If warnings were found, notify users. |
153
|
0
|
0
|
|
|
|
|
if ( $has_warnings ) |
154
|
|
|
|
|
|
|
{ |
155
|
|
|
|
|
|
|
# If we have a user, stop and ask if we should continue with the commit. |
156
|
|
|
|
|
|
|
# uncoverable branch true |
157
|
0
|
0
|
|
|
|
|
if ( $app->get_terminal()->is_interactive() ) |
158
|
|
|
|
|
|
|
{ |
159
|
0
|
|
|
|
|
|
print "Some warnings were found. Press to continue committing or Ctrl-C to abort the commit.\n"; |
160
|
0
|
|
|
|
|
|
my $input = ; ## no critic (InputOutput::ProhibitExplicitStdin) |
161
|
0
|
|
|
|
|
|
print "\n"; |
162
|
|
|
|
|
|
|
} |
163
|
|
|
|
|
|
|
# If we don't have a user, just warn and continue. |
164
|
|
|
|
|
|
|
else |
165
|
|
|
|
|
|
|
{ |
166
|
0
|
|
|
|
|
|
print "Some warnings were found, please review.\n"; |
167
|
|
|
|
|
|
|
} |
168
|
|
|
|
|
|
|
} |
169
|
|
|
|
|
|
|
|
170
|
0
|
|
|
|
|
|
return $tests_success; |
171
|
|
|
|
|
|
|
} |
172
|
|
|
|
|
|
|
|
173
|
|
|
|
|
|
|
|
174
|
|
|
|
|
|
|
=head1 BUGS |
175
|
|
|
|
|
|
|
|
176
|
|
|
|
|
|
|
Please report any bugs or feature requests through the web interface at |
177
|
|
|
|
|
|
|
L. |
178
|
|
|
|
|
|
|
I will be notified, and then you'll automatically be notified of progress on |
179
|
|
|
|
|
|
|
your bug as I make changes. |
180
|
|
|
|
|
|
|
|
181
|
|
|
|
|
|
|
|
182
|
|
|
|
|
|
|
=head1 SUPPORT |
183
|
|
|
|
|
|
|
|
184
|
|
|
|
|
|
|
You can find documentation for this module with the perldoc command. |
185
|
|
|
|
|
|
|
|
186
|
|
|
|
|
|
|
perldoc App::GitHooks::Hook::PreCommit |
187
|
|
|
|
|
|
|
|
188
|
|
|
|
|
|
|
|
189
|
|
|
|
|
|
|
You can also look for information at: |
190
|
|
|
|
|
|
|
|
191
|
|
|
|
|
|
|
=over |
192
|
|
|
|
|
|
|
|
193
|
|
|
|
|
|
|
=item * GitHub's request tracker |
194
|
|
|
|
|
|
|
|
195
|
|
|
|
|
|
|
L |
196
|
|
|
|
|
|
|
|
197
|
|
|
|
|
|
|
=item * AnnoCPAN: Annotated CPAN documentation |
198
|
|
|
|
|
|
|
|
199
|
|
|
|
|
|
|
L |
200
|
|
|
|
|
|
|
|
201
|
|
|
|
|
|
|
=item * CPAN Ratings |
202
|
|
|
|
|
|
|
|
203
|
|
|
|
|
|
|
L |
204
|
|
|
|
|
|
|
|
205
|
|
|
|
|
|
|
=item * MetaCPAN |
206
|
|
|
|
|
|
|
|
207
|
|
|
|
|
|
|
L |
208
|
|
|
|
|
|
|
|
209
|
|
|
|
|
|
|
=back |
210
|
|
|
|
|
|
|
|
211
|
|
|
|
|
|
|
|
212
|
|
|
|
|
|
|
=head1 AUTHOR |
213
|
|
|
|
|
|
|
|
214
|
|
|
|
|
|
|
L, |
215
|
|
|
|
|
|
|
C<< >>. |
216
|
|
|
|
|
|
|
|
217
|
|
|
|
|
|
|
|
218
|
|
|
|
|
|
|
=head1 COPYRIGHT & LICENSE |
219
|
|
|
|
|
|
|
|
220
|
|
|
|
|
|
|
Copyright 2013-2015 Guillaume Aubert. |
221
|
|
|
|
|
|
|
|
222
|
|
|
|
|
|
|
This program is free software: you can redistribute it and/or modify it under |
223
|
|
|
|
|
|
|
the terms of the GNU General Public License version 3 as published by the Free |
224
|
|
|
|
|
|
|
Software Foundation. |
225
|
|
|
|
|
|
|
|
226
|
|
|
|
|
|
|
This program is distributed in the hope that it will be useful, but WITHOUT ANY |
227
|
|
|
|
|
|
|
WARRANTY; without even the implied warranty of MERCHANTABILITY or FITNESS FOR A |
228
|
|
|
|
|
|
|
PARTICULAR PURPOSE. See the GNU General Public License for more details. |
229
|
|
|
|
|
|
|
|
230
|
|
|
|
|
|
|
You should have received a copy of the GNU General Public License along with |
231
|
|
|
|
|
|
|
this program. If not, see http://www.gnu.org/licenses/ |
232
|
|
|
|
|
|
|
|
233
|
|
|
|
|
|
|
=cut |
234
|
|
|
|
|
|
|
|
235
|
|
|
|
|
|
|
1; |