Re: Working on Pod-Simple

[James E Keenan]

On 03/05/2016 09:44 AM, Neil Bowers wrote:
> A few weeks ago I mailed that I was planning to work on improving things in modules at the head of the river. The first of these is Pod::Simple. I’m slowly working towards a PR with various changes, and doing developer releases (with Marc’s permission). I’ll outline the changes here, to give people a chance to comment, since the module has so many dists downstream.
>

Thanks for the update!

jimk

[Shlomi Fish]

On Sat, 5 Mar 2016 14:44:46 +0000
Neil Bowers <neil....@cogendo.com> wrote:

> A few weeks ago I mailed that I was planning to work on improving things in
> modules at the head of the river. The first of these is Pod::Simple. I’m
> slowly working towards a PR with various changes, and doing developer
> releases (with Marc’s permission). I’ll outline the changes here, to give
> people a chance to comment, since the module has so many dists downstream.
>

> Min perl version
> I’ve set min perl version to 5.006. It was previously not set in metadata,
> and all modules bar one had “require 5”. But the ChangeLog had discussion of
> changes that explicitly said anything earlier than 5.6 was no longer
> supported. If you look at CPAN Testers, you can see there are passes back to
> 5.6.2. http://matrix.cpantesters.org/?dist=Pod-Simple%203.32
>
> One module in the dist requires 5.008: Pod::Simple::TranscodeSmart. This is
> used by Pod::Simple::Transcode, which falls back onto using
> Pod::Simple::TranscodeDumb if it can’t load Pod::Simple::TranscodeSmart,
> which is why the dist passes tests on 5.6.2.
>
> So I set it to 5.006; another option would have been 5.006002 — since there
> is evidence back that far.
>
> Dropped “use vars”
> Dropped all uses of “use vars”, and replaced with “our” as appropriate.
>
> Add “use warnings”
> I added “use warnings” to all modules, and resolved the resulting warnings.
> Some things required appropriately scoped “no warnings …”
>
> Fixed CPAN Testers fails
> As you can see from the CPAN Testers matrix link above, the current stable
> release has various failures. I’ve fixed most of them, and my current dev
> release has one particular failing test, that I’m hoping to sort out this
> weekend: http://matrix.cpantesters.org/?dist=Pod-Simple
> <http://matrix.cpantesters.org/?dist=Pod-Simple> This is one of my main
> goals: to get this green across the board.
>
> Neil
>

thanks Neil for your contributions!

-- Shlomi