Skip to content
This repository was archived by the owner on Nov 1, 2017. It is now read-only.

Update content/v3/repos/forks.md#200

Closed
mattclegg wants to merge 1 commit into
github:masterfrom
mattclegg:repos-forks
Closed

Update content/v3/repos/forks.md#200
mattclegg wants to merge 1 commit into
github:masterfrom
mattclegg:repos-forks

Conversation

@mattclegg

Copy link
Copy Markdown

Just to make it explicit as its not clear on first inspection.

Just to make it explicit as its not clear on first inspection.
@pengwynn

Copy link
Copy Markdown
Contributor

Thanks for the patch, but I'm not sure we need to treat this section as special. These resources follow the convention for RESTful resource URLs used by the entire API. Perhaps the HTTP Verbs section would be a better fit.

@pengwynn pengwynn closed this Jan 27, 2013
@mattclegg

Copy link
Copy Markdown
Author

Thanks, Id missed that.

On Sun, Jan 27, 2013 at 1:40 PM, Wynn Netherland
[email protected]:

Thanks for the patch, but I'm not sure we need to treat this section as
special. These resources follow the convention for RESTful resource URLshttp://en.wikipedia.org/wiki/Representational_state_transfer#RESTful_web_servicesused by the entire API. Perhaps the HTTP
Verbs http://developer.github.com/v3/#http-verbs section would be a
better fit.


Reply to this email directly or view it on GitHubhttps://github.com//pull/200#issuecomment-12754216.

Matt Clegg

--In January Im giving up my favourite food for cancer research.
http://www.justgiving.com/dryathlete-matt-clegg

@mattclegg mattclegg deleted the repos-forks branch January 27, 2013 13:48
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