Add Test::Pod test dependency #19

Merged
merged 1 commit into from Nov 10, 2016

Conversation

Projects
None yet
4 participants
@paultcochrane
Contributor

paultcochrane commented Nov 9, 2016

After having built a pristine Perl with perlbrew it turned out that
after having installed all the dependencies via cpanm --installdeps .
that the Test::Pod module was still required in order to run dzil test. This patch adds Test::Pod to the list of test dependencies so
that this issue doesn't happen to someone else.

If you need this PR changed in any way, please just let me know and I'll update it and resubmit.

Add Test::Pod test dependency
After having built a pristine Perl with perlbrew it turned out that
after having installed all the dependencies via `cpanm --installdeps .`
that the `Test::Pod` module was still required in order to run `dzil
test`.  This patch adds `Test::Pod` to the list of test dependencies so
that this issue doesn't happen to someone else.
@coveralls

This comment has been minimized.

Show comment
Hide comment
@coveralls

coveralls Nov 9, 2016

Coverage Status

Coverage remained the same at 99.412% when pulling 913298c on paultcochrane:pr/add-test-prereq into bdd9125 on fayland:master.

Coverage Status

Coverage remained the same at 99.412% when pulling 913298c on paultcochrane:pr/add-test-prereq into bdd9125 on fayland:master.

@coveralls

This comment has been minimized.

Show comment
Hide comment
@coveralls

coveralls Nov 9, 2016

Coverage Status

Coverage remained the same at 99.412% when pulling 913298c on paultcochrane:pr/add-test-prereq into bdd9125 on fayland:master.

coveralls commented Nov 9, 2016

Coverage Status

Coverage remained the same at 99.412% when pulling 913298c on paultcochrane:pr/add-test-prereq into bdd9125 on fayland:master.

@codecov-io

This comment has been minimized.

Show comment
Hide comment
@codecov-io

codecov-io Nov 9, 2016

Current coverage is 95.88% (diff: 100%)

Merging #19 into master will not change coverage

@@             master        #19   diff @@
==========================================
  Files            11         11          
  Lines           170        170          
  Methods           0          0          
  Messages          0          0          
  Branches         17         17          
==========================================
  Hits            163        163          
  Misses            1          1          
  Partials          6          6          

Powered by Codecov. Last update bdd9125...913298c

codecov-io commented Nov 9, 2016

Current coverage is 95.88% (diff: 100%)

Merging #19 into master will not change coverage

@@             master        #19   diff @@
==========================================
  Files            11         11          
  Lines           170        170          
  Methods           0          0          
  Messages          0          0          
  Branches         17         17          
==========================================
  Hits            163        163          
  Misses            1          1          
  Partials          6          6          

Powered by Codecov. Last update bdd9125...913298c

@fayland fayland merged commit d80327d into fayland:master Nov 10, 2016

1 check passed

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

This comment has been minimized.

Show comment
Hide comment
@fayland

fayland Nov 10, 2016

Owner

thanks.

Owner

fayland commented Nov 10, 2016

thanks.

@paultcochrane paultcochrane deleted the paultcochrane:pr/add-test-prereq branch Nov 10, 2016

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