Skip to content

Commit 8be7c64

Browse files
committed
indexer: drop old warning code, use the new code
1 parent 7651959 commit 8be7c64

File tree

1 file changed

+3
-20
lines changed

1 file changed

+3
-20
lines changed

lib/PAUSE/dist.pm

Lines changed: 3 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -422,7 +422,9 @@ sub mail_summary {
422422
"The following report has been written by the PAUSE namespace indexer.\n",
423423
"Please contact modules\@perl.org if there are any open questions.\n";
424424

425-
if ($self->has_indexing_warnings($ctx)) {
425+
if ($ctx->warnings_for_all_packages) {
426+
# If there were any warnings, put in a note to the reader that they should
427+
# look for them.
426428
push @m,
427429
"\nWARNING: Some irregularities were found while indexing your\n",
428430
" distribution. See below for more details.\n";
@@ -567,19 +569,6 @@ sub index_status {
567569
};
568570
}
569571

570-
sub indexing_warnings_for_package {
571-
my ($self, $ctx, $pack) = @_;
572-
return @{ $self->{INDEX_WARNINGS}{$pack} // [] };
573-
}
574-
575-
sub has_indexing_warnings {
576-
my ($self, $ctx) = @_;
577-
my $i;
578-
my $warnings = $self->{INDEX_WARNINGS};
579-
580-
@$_ && return 1 for values %$warnings;
581-
}
582-
583572
sub check_blib {
584573
my ($self, $ctx) = @_;
585574
if (grep m|^[^/]+/blib/|, @{$self->{MANIFOUND}}) {
@@ -1275,12 +1264,6 @@ Accessor method. True if perl distro from non-pumpking or a dev release.
12751264
12761265
=head3 index_status
12771266
1278-
=head3 add_indexing_warning
1279-
1280-
=head3 indexing_warnings_for_package
1281-
1282-
=head3 has_indexing_warnings
1283-
12841267
=head3 _package_governing_permission
12851268
12861269
The package used to determine whether the uploader may upload this distro.

0 commit comments

Comments
 (0)