Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Update readmes with three fixes #106

Closed
wants to merge 3 commits into
from

Conversation

Projects
None yet
2 participants
Contributor

cldwalker commented Oct 29, 2011

Hi,
These three fixes tripped me up while creating a hubot. At least one fix would clarify setup for another user (#102). Thanks for the awesome project!

cldwalker added some commits Oct 29, 2011

@cldwalker cldwalker update readme to use bin/hubot correctly
otherwise users have to resort to installing coffee-script globally
for example hubotio/hubot#102
3bf1e04
@cldwalker cldwalker point to a correct README, there is none in hubot/ 0d20986
@cldwalker cldwalker update readme for heroku version to use hubot -c
I'm assuming hubot -c is desired since that's the only
way the scripts/ reference makes sense
78cf56e

@atmos atmos commented on the diff Oct 29, 2011

src/templates/README.md
@@ -8,9 +8,17 @@ This version is designed to be deployed on heroku.
Playing with Hubot
==================
-You'll need to install the necessary dependencies for hubot. All of
+First, create your own hubot, nubot.
+
+ % npm install
+ % PATH="node_modules/.bin:$PATH" bin/hubot -c nubot
@atmos

atmos Oct 29, 2011

Contributor

This is too much for the tagged releases. All of this should be wrapped up in bin/hubot. People shouldn't be using bin/hubot -c.

@atmos atmos closed this in 09e1a50 Oct 29, 2011

@glucero glucero pushed a commit to absperf/hubot that referenced this pull request Aug 20, 2012

@atmos atmos update PATH for local testing, fixes #106 4f19ff8
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment