Skip to content

Add a step to the RMG about podcheck #20975

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
khwilliamson wants to merge 2 commits into from
Closed

Add a step to the RMG about podcheck #20975

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
11cd323
Add a step to the RMG about podcheck
khwilliamson Oct 22, 2022
fce9f55
Merge branch 'Perl:blead' into rmg
khwilliamson Mar 31, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
15 changes: 15 additions & 0 deletions Porting/release_managers_guide.pod
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -397,6 +397,21 @@ colon-delimited versions to use for comparison. For example:

L<http://analysis.cpantesters.org/beforemaintrelease?pair=5.20.2:5.22.0%20RC1>

=head3 check pod errors

F<t/porting/podcheck.t> is a porting test that will fail if it finds new
problems in pods. However, it can be taught to ignore problems, and
sometimes people do so for problems that really should be fixed before
release. To see what it is ignoring, run

perl t/porting/podcheck.t --counts

Any problems listed as pedantic aren't worth your time investigating.
These have a C<?> at the beginning of the text, or are for the too-long
verbatim lines.

But other warnings should be. In particular, a broken link can well
mean that someone clicking on the pod in a web page will get a 404.
Copy link
Collaborator

@demerphq demerphq Mar 28, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There should be a blank line after this one and before the next =head3

Copy link
Contributor Author

@khwilliamson khwilliamson Mar 31, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks

=head3 update perldelta

Get perldelta in a mostly finished state.
Expand Down