Skip to content
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

Added basic testing framework #9

Closed
wants to merge 1 commit into from
Closed

Added basic testing framework #9

wants to merge 1 commit into from

Conversation

solarkennedy
Copy link

This adds the basic boilerplate to test this module with:

bundle install
bundle exec rake spec

It also includes .travis stuff which integrates well with Github.

I also changed the install class from "package" to "install" and keyed off of $::osfamily instead of $::operatingsystem to conform to more standard module best practice stuff. (hope that is ok)

Closes #7.

@solarkennedy
Copy link
Author

Bump? If you are not interested I will simple close the PR and leave it on my own fork.

@akumria
Copy link
Owner

akumria commented Mar 22, 2015

Sorry!

It looks like github defaulted notifications off (at work I use bitbucket so rarely login here).

I will take a look, and either comment (or merge) tonight.

@akumria
Copy link
Owner

akumria commented Mar 31, 2015

This all looks good, thanks.

I've merged locally and will push out a release this week with your changes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Tests PR?
2 participants