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

Require App::CPANTS::Lint for kwalitee checks #26

Merged
merged 1 commit into from Oct 29, 2017

Conversation

Projects
None yet
3 participants
@paultcochrane
Contributor

paultcochrane commented Oct 16, 2017

The cpants_lint.pl program has been extracted into a separate project
and hence one needs App::CPANTS::Lint in order that the kwalitee tests
in Module::Release can be run. Here I've extracted the required
module names into variables to try and be more descriptive, but also to
reduce the length of the strings required in the eval and in the die
message.

Require App::CPANTS::Lint for kwalitee checks
The cpants_lint.pl program has been extracted into a separate project
and hence one needs `App::CPANTS::Lint` in order that the kwalitee tests
in `Module::Release` can be run.  Here I've extracted the required
module names into variables to try and be more descriptive, but also to
reduce the length of the strings required in the `eval` and in the `die`
message.
@coveralls

This comment has been minimized.

Show comment
Hide comment
@coveralls

coveralls Oct 16, 2017

Coverage Status

Coverage increased (+0.1%) to 67.076% when pulling b6adec2 on paultcochrane:pr/require-app-cpants-lint into 6532dbf on briandfoy:master.

coveralls commented Oct 16, 2017

Coverage Status

Coverage increased (+0.1%) to 67.076% when pulling b6adec2 on paultcochrane:pr/require-app-cpants-lint into 6532dbf on briandfoy:master.

@briandfoy briandfoy merged commit a6a9a64 into briandfoy:master Oct 29, 2017

2 checks passed

continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details

@paultcochrane paultcochrane deleted the paultcochrane:pr/require-app-cpants-lint branch Nov 1, 2017

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