Perl / docker-perl-tester

Docker images with pre-installed test modules and test dependencies for CPAN modules
28 stars 14 forks source link

Provide App::perlimports to the image #60

Open bschmalhofer opened 7 months ago

bschmalhofer commented 7 months ago

Hi,

I am fond of perlimports for cleaning up my import statements. Would it make sense to add that module to the image? The list of dependencies looks a bit daunting though.

Edit: The issue with DBD::ODBC was unrelated.

atoomic commented 7 months ago

I see no objections to add it, but also note that you can add extra packages using a cpanfile with cpm or cpanm actions

Note: there is one example described here https://github.com/Perl/docker-perl-tester?tab=readme-ov-file#using-the-images-with-github-workflow and the two actions are documented there:

oalders commented 7 months ago

I am fond of perlimports for cleaning up my import statements

❤️

We could add it for perl >= '5.18.0'. It would pull in a lot of dependencies, but I'm not sure how bad that would be, considering how many dependencies these builds already contain.

bschmalhofer commented 7 months ago

I see no objections to add it, but also note that you can add extra packages using a cpanfile with cpm or cpanm actions

Yes, that is what I do in https://github.com/RotherOSS/otobo/blob/5c899d77c7d9e33b9471ca26fcbfbc60a9ed68d1/.github/workflows/code_policy.yml#L60 . This works fine, so adding App::perlimports would be just for convenience.