Skip to content
This repository was archived by the owner on Aug 20, 2018. It is now read-only.

Refactor command #5

Closed
wants to merge 10 commits into from
Closed

Conversation

gfmurphy
Copy link

I'm just sketching some ideas with the main idea being that the call to the api is encapsulated in a command which allows for easy manipulation before the command is executed. This is done by yielding the command to the block that is passed to the command method.

Let me know what you guys think.

@sirupsen
Copy link
Contributor

I definitely like your ideas here, I will check it out properly when I return from holiday. Your contribution is greatly appreciated!

@gfmurphy
Copy link
Author

Thanks! Enjoy your holiday!

George

Sent from my iPhone

On Jul 13, 2011, at 9:26 AM, Sirupsenreply@reply.github.com wrote:

I definitely like your ideas here, I will check it out properly when I return from holiday. Your contribution is gratly appreciated!

Reply to this email directly or view it on GitHub:
#5 (comment)

@gfmurphy gfmurphy closed this May 5, 2018
@gfmurphy gfmurphy deleted the refactor_command branch May 5, 2018 18:12
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants