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

Updated hubot initialization #32

Closed
wants to merge 3 commits into from
Closed

Conversation

frozenfoxx
Copy link

This is intended to fix issue #30. I've altered the installation to make sure yeoman and the hubot generator are installed if they are not already, then modified the config.pp to use the new yeoman method of installation.

The path is longer for more accomodation but in the future this really should be a variable set on operating system.

Added ensuring resources "yo" and "generator-hubot" to update functionality.
Fixed the Hubot init exec statement for updated hubot functionality.
@frozenfoxx
Copy link
Author

frozenfoxx commented May 20, 2016

Hmm, apparently the tests are still looking for the "hubot -c" directly which we're trying to remove since it's no longer supported. I'll look into it.

Adjusted the command to reflect what the current command output should be.
@frozenfoxx
Copy link
Author

It seems the rake gem doesn't like Ruby older than the 1.9.x branch. Do we care about Ruby 1.8.7 anymore? If so is there a way to set the rake version to the newest supported by the 1.8.7 Ruby version? I'd do it myself but I'm really not sure how to do that and all new versions of Puppet include their own Ruby that's significantly newer.

@frozenfoxx
Copy link
Author

frozenfoxx commented May 23, 2016

Need to close and reopen as Travis is not picking up my latest commit for some reason. Continued here.

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.

Yeoman hubot initiation.
1 participant