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.

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

Add missing File::Temp prerequisite #20

Closed
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
2 participants
@paultcochrane
Contributor

paultcochrane commented Apr 23, 2017

This solves the 'prereq matches use'
CPANTS issue for this
distribution.

I haven't set a definite File::Temp version since it's not clear to me what the version really should be (and is probably more obvious to someone more familiar with the dist). Nevertheless, this change fixes a CPANTS issue and, if a better File::Temp version can be used, then will hopefully provide an improvement to the dist.

If you wish for the PR to be changed in any way, please just let me know and I'll update and resubmit.

Add missing File::Temp prerequisite
This solves the 'prereq matches use'
[CPANTS](http://cpants.cpanauthors.org/dist/Test-Script) issue for this
distribution.
@plicease

This comment has been minimized.

Show comment
Hide comment
@plicease

plicease May 1, 2017

Owner

I see the rationale for including core modules as they may be removed from the core in the future. On the other hand I find dists that specify every core module get very noisy in their prereqs, which is why I tend not to include them. I feel moderately safe that File::Temp won't be removed from core, so I don't think I will merge this one.

Owner

plicease commented May 1, 2017

I see the rationale for including core modules as they may be removed from the core in the future. On the other hand I find dists that specify every core module get very noisy in their prereqs, which is why I tend not to include them. I feel moderately safe that File::Temp won't be removed from core, so I don't think I will merge this one.

@plicease plicease closed this May 1, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment