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

Refactor request body validation #136

Merged
merged 2 commits into from
May 26, 2014
Merged

Conversation

ixti
Copy link
Member

@ixti ixti commented May 25, 2014

No description provided.

@ixti
Copy link
Member Author

ixti commented May 25, 2014

@tarcieri Ready for review/merge ;))

@sferik
Copy link
Contributor

sferik commented May 25, 2014

👍 Looks good to me.

@Asmod4n
Copy link
Contributor

Asmod4n commented May 26, 2014

Shouldn't a Socket/IO also be supported?

@ixti
Copy link
Member Author

ixti commented May 26, 2014

@Asmod4n IO includes Enumerable, and Socket is a descendant of IO, In other words both Socket and IO instances are is_a? Enumerable :D

@ixti
Copy link
Member Author

ixti commented May 26, 2014

@tarcieri If no objections I can merge :D

@tarcieri
Copy link
Member

@ixti 👍 from me

ixti added a commit that referenced this pull request May 26, 2014
@ixti ixti merged commit ce1fa3f into master May 26, 2014
@ixti ixti deleted the refactor/request-body-validation branch May 26, 2014 22:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants