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

Initialize the App class .instance is called on #33

Merged
merged 1 commit into from
Dec 7, 2015

Conversation

dmvt
Copy link
Contributor

@dmvt dmvt commented Dec 1, 2015

Initializes the subclass (for instance PongBot::App in the first README example) instead of SlackRubyBot::App specifically. This allows for easier behavior modification through custom Hooks and direct method overriding.

@dmvt dmvt changed the title Initialize the App class #instance is called on Initialize the App class .instance is called on Dec 1, 2015
@dblock
Copy link
Collaborator

dblock commented Dec 2, 2015

Very nice. I'd love a CHANGELOG entry and a test for this, please?

@dmvt
Copy link
Contributor Author

dmvt commented Dec 2, 2015

happy to... will update this evening

@dblock
Copy link
Collaborator

dblock commented Dec 7, 2015

Bump @dmvt, want to cut a release soon would be nice to get this.

@dmvt
Copy link
Contributor Author

dmvt commented Dec 7, 2015

Sorry I missed the cut

@@ -1,5 +1,6 @@
### 0.5.1 (Next)

* [#33](https://github.com/dblock/slack-ruby-bot/pull/33): `SlackRubyBot::App.instance` now creates an instance of the class on which it is called.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Make this look like all the other changelog lines with a link to your Github profile.

@dblock
Copy link
Collaborator

dblock commented Dec 7, 2015

No big deal, next cut. Can you squash your commits, too, please.

Initializes the subclass (for instance `PongBot::App` in the first `README` example) instead of `SlackRubyBot::App` specifically. This allows for easier behavior modification through custom `Hooks` and direct method overriding.
@dmvt
Copy link
Contributor Author

dmvt commented Dec 7, 2015

All set :)

@dblock
Copy link
Collaborator

dblock commented Dec 7, 2015

👍

dblock added a commit that referenced this pull request Dec 7, 2015
Initialize the App class .instance is called on
@dblock dblock merged commit bcb5096 into slack-ruby:master Dec 7, 2015
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.

None yet

2 participants