Remove Test::More from Travis config #22

Merged
merged 1 commit into from Nov 11, 2016

Conversation

Projects
None yet
4 participants
@paultcochrane
Contributor

paultcochrane commented Nov 10, 2016

Test::More is in core, so shouldn't need to be explicitly installed as
part of the Travis build.

The Travis tests still pass, however if it was intended to keep Test::More here, then please feel free to close the PR unmerged.

Remove Test::More from Travis config
`Test::More` is in core, so shouldn't need to be explicitly installed as
part of the Travis build.
@coveralls

This comment has been minimized.

Show comment
Hide comment
@coveralls

coveralls Nov 10, 2016

Coverage Status

Coverage remained the same at 99.492% when pulling 2af3b6f on paultcochrane:pr/remove-test-more-from-travis into ecffd69 on fayland:master.

coveralls commented Nov 10, 2016

Coverage Status

Coverage remained the same at 99.492% when pulling 2af3b6f on paultcochrane:pr/remove-test-more-from-travis into ecffd69 on fayland:master.

@codecov-io

This comment has been minimized.

Show comment
Hide comment
@codecov-io

codecov-io Nov 10, 2016

Current coverage is 93.40% (diff: 100%)

Merging #22 into master will not change coverage

@@             master        #22   diff @@
==========================================
  Files            14         14          
  Lines           197        197          
  Methods           0          0          
  Messages          0          0          
  Branches         26         26          
==========================================
  Hits            184        184          
  Misses            1          1          
  Partials         12         12          

Powered by Codecov. Last update ecffd69...2af3b6f

codecov-io commented Nov 10, 2016

Current coverage is 93.40% (diff: 100%)

Merging #22 into master will not change coverage

@@             master        #22   diff @@
==========================================
  Files            14         14          
  Lines           197        197          
  Methods           0          0          
  Messages          0          0          
  Branches         26         26          
==========================================
  Hits            184        184          
  Misses            1          1          
  Partials         12         12          

Powered by Codecov. Last update ecffd69...2af3b6f

@fayland fayland merged commit 404762d into fayland:master Nov 11, 2016

1 check passed

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