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

replace Class::Load by Module::Runtime #12

Merged
merged 1 commit into from Sep 27, 2016

Conversation

Projects
None yet
2 participants
@yanick
Contributor

yanick commented Sep 25, 2016

A very minor tweak. Since Class::Load uses Module::Runtime under the hood, going straight for M::Rt knocks one module off the dependency list.

(I got this module as my CPAN Challenge monthly task, and oh my is it hard to find something to improve in this module. Everything is ever so very nicely polished already :-) )

replace Class::Load by Module::Runtime
A very minor tweak. Since Class::Load uses Module::Runtime
under the hood, going straight for M::Rt knocks one
module off the dependency list.
@oalders

This comment has been minimized.

Show comment
Hide comment
@oalders

oalders Sep 27, 2016

Member

Great! Thanks very much for this @yanick. :)

Member

oalders commented Sep 27, 2016

Great! Thanks very much for this @yanick. :)

@oalders oalders merged commit d175128 into libwww-perl:master Sep 27, 2016

1 check failed

continuous-integration/travis-ci/pr The Travis CI build could not complete due to an error
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment