Add missing prereqs to match use #21

Merged
merged 1 commit into from Nov 10, 2016

Conversation

Projects
None yet
4 participants
@paultcochrane
Contributor

paultcochrane commented Nov 10, 2016

The kwalitee tests on
CPANTS
show that some modules are used but not declared as prerequisites. This
change brings code up to date and thus closes the prereq_matches_use
CPANTS core issue for this distribution.

Add missing prereqs to match use
The kwalitee tests on
[CPANTS](http://cpants.cpanauthors.org/dist/Dist-Zilla-Plugin-GitHubREADME-Badge)
show that some modules are used but not declared as prerequisites.  This
change brings code up to date and thus closes the prereq_matches_use
CPANTS core issue for this 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 ae8d27a on paultcochrane:pr/add-missing-prereqs into 30727f0 on fayland:master.

coveralls commented Nov 10, 2016

Coverage Status

Coverage remained the same at 99.492% when pulling ae8d27a on paultcochrane:pr/add-missing-prereqs into 30727f0 on fayland:master.

@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 ae8d27a on paultcochrane:pr/add-missing-prereqs into 30727f0 on fayland:master.

Coverage Status

Coverage remained the same at 99.492% when pulling ae8d27a on paultcochrane:pr/add-missing-prereqs into 30727f0 on fayland:master.

@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 ae8d27a on paultcochrane:pr/add-missing-prereqs into 30727f0 on fayland:master.

Coverage Status

Coverage remained the same at 99.492% when pulling ae8d27a on paultcochrane:pr/add-missing-prereqs into 30727f0 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 #21 into master will not change coverage

@@             master        #21   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 30727f0...ae8d27a

codecov-io commented Nov 10, 2016

Current coverage is 93.40% (diff: 100%)

Merging #21 into master will not change coverage

@@             master        #21   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 30727f0...ae8d27a

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

I'll ship a version today. thanks

Owner

fayland commented Nov 10, 2016

I'll ship a version today. thanks

@paultcochrane

This comment has been minimized.

Show comment
Hide comment
@paultcochrane

paultcochrane Nov 10, 2016

Contributor

My pleasure! Anything else I can help out with?

Contributor

paultcochrane commented Nov 10, 2016

My pleasure! Anything else I can help out with?

@paultcochrane paultcochrane deleted the paultcochrane:pr/add-missing-prereqs branch Nov 10, 2016

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