Add prereq missing from build_requires #24

Merged
merged 1 commit into from Nov 11, 2016

Conversation

Projects
None yet
4 participants
@paultcochrane
Contributor

paultcochrane commented Nov 10, 2016

The experimental CPANTS metric build_prereq_matches_use shows that the
warnings prereq is missing from build_requires. By adding this
prereq it is now very explicit what packages are required to build and
test the distribution.

Add prereq missing from build_requires
The experimental CPANTS metric `build_prereq_matches_use` shows that the
`warnings` prereq is missing from `build_requires`.  By adding this
prereq it is now very explicit what packages are required to build and
test the distribution.
@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 16d8a75 on paultcochrane:pr/add-missing-build-requires-prereqs into ecffd69 on fayland:master.

coveralls commented Nov 10, 2016

Coverage Status

Coverage remained the same at 99.492% when pulling 16d8a75 on paultcochrane:pr/add-missing-build-requires-prereqs 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 #24 into master will not change coverage

@@             master        #24   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...16d8a75

codecov-io commented Nov 10, 2016

Current coverage is 93.40% (diff: 100%)

Merging #24 into master will not change coverage

@@             master        #24   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...16d8a75

@fayland fayland merged commit 63a42f9 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