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

Have distmanifest.t check for test module first. #22

Closed
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
2 participants
@adherzog
Contributor

adherzog commented Feb 24, 2015

No description provided.

@adherzog

This comment has been minimized.

Contributor

adherzog commented Feb 24, 2015

(Fixes #21)

@adherzog

This comment has been minimized.

Contributor

adherzog commented Feb 24, 2015

It looks like @karenetheridge fixed the original problem. This is still a nice addition, for anyone who might be getting URI from the git repo but doesn't want to run the author tests.

@karenetheridge

This comment has been minimized.

Member

karenetheridge commented Jun 26, 2015

closing - I think anyone hacking on this distribution should have to run the manifest test. The test module doesn't have many prereqs so this shouldn't be onerous (or, just let the test fail if you don't want to install it).

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