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.

Sign up for GitHub

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

Jump to bottom

Add bot announce #12

Closed
wants to merge 4 commits into from
Closed

Add bot announce #12

wants to merge 4 commits into from

Conversation

jm-welch
Copy link
Contributor

Fixes #10

@jm-welch
Copy link
Contributor Author

Bot will announce its presence if HUBOT_ANNOUNCE environment variable is set, but take no additional action if it is not set.

@splunk-cwanek
Copy link
Contributor

Will you please bump the version as well?

@jm-welch
Update readme
3c77b13 
Cover HUBOT_ANNOUNCE var
@jm-welch
Bump version
daa4e48
@jm-welch
Copy link
Contributor Author

Version done bumped and readme updated to include the new env var.

@splunk-cwanek
Copy link
Contributor

Actually, the version should be .13 (or .14 if you're superstitious).

@jm-welch
Copy link
Contributor Author

Not sure what conflict this is reporting now.

@jm-welch
Copy link
Contributor Author

Closing this - will resubmit this and a fix for #14 as one pull request, without the conflict.

@jm-welch jm-welch closed this Jul 20, 2016
@jm-welch jm-welch deleted the jm-welch-announce branch July 20, 2016 19:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jm-welch @splunk-cwanek