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

Update perlcritic.t #2

Merged
merged 1 commit into from Jan 31, 2015

Conversation

Projects
None yet
3 participants
@jscook
Contributor

jscook commented Jan 31, 2015

removed unused module

Update perlcritic.t
removed unused module
@coveralls

This comment has been minimized.

Show comment
Hide comment
@coveralls

coveralls Jan 31, 2015

Coverage Status

Coverage remained the same at 100.0% when pulling 0c970c5 on jscook:patch-1 into 0d32518 on guillaumeaubert:master.

coveralls commented Jan 31, 2015

Coverage Status

Coverage remained the same at 100.0% when pulling 0c970c5 on jscook:patch-1 into 0d32518 on guillaumeaubert:master.

@coveralls

This comment has been minimized.

Show comment
Hide comment
@coveralls

coveralls Jan 31, 2015

Coverage Status

Coverage remained the same at 100.0% when pulling 0c970c5 on jscook:patch-1 into 0d32518 on guillaumeaubert:master.

coveralls commented Jan 31, 2015

Coverage Status

Coverage remained the same at 100.0% when pulling 0c970c5 on jscook:patch-1 into 0d32518 on guillaumeaubert:master.

guillaumeaubert added a commit that referenced this pull request Jan 31, 2015

@guillaumeaubert guillaumeaubert merged commit 586adbd into guillaumeaubert:master Jan 31, 2015

1 check passed

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

guillaumeaubert added a commit that referenced this pull request Jan 31, 2015

@guillaumeaubert

This comment has been minimized.

Show comment
Hide comment
@guillaumeaubert

guillaumeaubert Jan 31, 2015

Owner

Thank you for spotting this! This completely removes File::Spec from the dependencies for this module, and I've rebuilt the meta files accordingly.

I'm pretty sure this problem affects all my other distributions as well (since I keep all the xt/ tests in sync), so I'll review them next.

Owner

guillaumeaubert commented Jan 31, 2015

Thank you for spotting this! This completely removes File::Spec from the dependencies for this module, and I've rebuilt the meta files accordingly.

I'm pretty sure this problem affects all my other distributions as well (since I keep all the xt/ tests in sync), so I'll review them next.

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