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

added check for more dependencies to test 000-load.t #9

Closed
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
2 participants
@viviparous

viviparous commented Jun 29, 2015

Perl Pull Request Challenge
I was given your module this month. The code is clean -- the only thing I could think to add was the checks for libraries needed for testing.

@rjbs

This comment has been minimized.

Show comment
Hide comment
@rjbs

rjbs Jul 1, 2015

Owner

Thanks, but this isn't the sort of thing I'd like my tests to be doing. If the prereqs can't be loaded, the problem will show in other ways.

In July, consider checking out the Issues list for your assignment. For example, about a week ago, someone filed #3 which could've been a pretty simple PR. :)

Owner

rjbs commented Jul 1, 2015

Thanks, but this isn't the sort of thing I'd like my tests to be doing. If the prereqs can't be loaded, the problem will show in other ways.

In July, consider checking out the Issues list for your assignment. For example, about a week ago, someone filed #3 which could've been a pretty simple PR. :)

@rjbs rjbs closed this Jul 1, 2015

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