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

Made test dependencies more lenient #12

Merged
merged 1 commit into from Feb 16, 2015

Conversation

Projects
None yet
3 participants
@geofffranks
Contributor

geofffranks commented Feb 7, 2015

  • Added a Perl::Critic test, only run if Test::Perl::Critic
    is installed.
  • Ensured the pod + pod coverage tests only run if Test::Pod
    and Test::Pod::Coverage are installed.
  • Removed dependencies for Test::Pod, Test::Pod::Coverage, and
    Test::Perl::Critic, since testing no longer explicitly requires
    them.

fixes #8

Made test dependencies more lenient
* Added a Perl::Critic test, only run if Test::Perl::Critic
  is installed.
* Ensured the pod + pod coverage tests only run if Test::Pod
  and Test::Pod::Coverage are installed.
* Removed dependencies for Test::Pod, Test::Pod::Coverage, and
  Test::Perl::Critic, since testing no longer explicitly requires
  them.

fixes #8

hakamadare added a commit that referenced this pull request Feb 16, 2015

Merge pull request #12 from geofffranks/testing-reqs
Made test dependencies more lenient

@hakamadare hakamadare merged commit b567292 into hakamadare:master Feb 16, 2015

1 check passed

continuous-integration/travis-ci The Travis CI build passed
Details
@frioux

This comment has been minimized.

Show comment
Hide comment
@frioux

frioux Feb 16, 2015

FWIW Moo already turns on strict.

frioux commented on lib/WebService/Pushover.pm in bb36dad Feb 16, 2015

FWIW Moo already turns on strict.

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