Emoji config #2

Merged
merged 5 commits into from Feb 20, 2013

3 participants

@linedotstar

[Reviewer: @jcwilk ]

@jcwilk jcwilk and 1 other commented on an outdated diff Feb 20, 2013
lib/capistrano/campystrano.rb
@@ -14,12 +14,13 @@ def self.load_into(configuration)
_cset(:campy_application) { fetch(:application) }
_cset(:campy_app_env) { fetch(:rails_env) }
_cset(:campy_branch) { fetch(:branch) rescue '' }
+ _cset(:campy_emoji) { (fetch(:campfire_emoji) rescue ':sparkles:') }
@jcwilk
jcwilk added a note Feb 20, 2013

extra parens here

they aren't needed to scope the rescue?

nvm

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
@jcwilk jcwilk and 1 other commented on an outdated diff Feb 20, 2013
-In your ```config``` directory, add a ```campystrano.yml``` file with your Campfire settings.
+The ```:campfire_settings``` variable must be defined as a hash containing your Campfire account credentials. The ```:subdomain``` and ```:room``` are required. You must also set either a ```:token``` or a ```:username```/```:password``` pair.
+
+For example:
```
@jcwilk
jcwilk added a note Feb 20, 2013

gotta rep the new emoji config var! :D devs love that shit like catnip

easter 🍳

@jcwilk
jcwilk added a note Feb 20, 2013

hehe very well

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

looks good ^_^

Chris Friedrich added some commits Feb 19, 2013
Chris Friedrich style fix b876b35
Chris Friedrich READ.me clarifications 5bef8e9
Chris Friedrich emoji readme update 2d26f57
Chris Friedrich Merge branch 'master' of github.com:lumoslabs/campystrano into emoji_…
…config

Conflicts:
	README.md
0fd2b75
@linedotstar linedotstar merged commit 00cce12 into master Feb 20, 2013
@linedotstar linedotstar deleted the emoji_config branch Feb 20, 2013
@azach

Small typo: set :campfire_emoji

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