Skip to content

Conversation

@emmanuel
Copy link
Contributor

No description provided.

@seancribbs
Copy link
Member

Looks good to me. +1 to merge if all specs pass.

@seancribbs
Copy link
Member

One more thing, please document all new methods and constants using YARD syntax.

@emmanuel
Copy link
Contributor Author

I added YARD documentation for the methods and then added Request#https? to test if a given request was made via HTTPS (since @bernerdschaefer mentioned that test in #25, which was the inspiration for this).

emmanuel added a commit that referenced this pull request Jan 16, 2012
Add Request verb queries for standard HTTP verbs
@emmanuel emmanuel merged commit b7ec78d into webmachine:master Jan 16, 2012
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.

2 participants