{"action":"created","issue":{"url":"https://api.github.com/repos/there4/slim-test-helpers/issues/19","labels_url":"https://api.github.com/repos/there4/slim-test-helpers/issues/19/labels{/name}","comments_url":"https://api.github.com/repos/there4/slim-test-helpers/issues/19/comments","events_url":"https://api.github.com/repos/there4/slim-test-helpers/issues/19/events","html_url":"https://github.com/there4/slim-test-helpers/pull/19","id":97672517,"number":19,"title":"Replace __call with explicit method declarations","user":{"login":"adbre","id":1292956,"avatar_url":"https://avatars.githubusercontent.com/u/1292956?v=3","gravatar_id":"","url":"https://api.github.com/users/adbre","html_url":"https://github.com/adbre","followers_url":"https://api.github.com/users/adbre/followers","following_url":"https://api.github.com/users/adbre/following{/other_user}","gists_url":"https://api.github.com/users/adbre/gists{/gist_id}","starred_url":"https://api.github.com/users/adbre/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/adbre/subscriptions","organizations_url":"https://api.github.com/users/adbre/orgs","repos_url":"https://api.github.com/users/adbre/repos","events_url":"https://api.github.com/users/adbre/events{/privacy}","received_events_url":"https://api.github.com/users/adbre/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-07-28T10:38:28Z","updated_at":"2015-07-28T15:15:13Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/there4/slim-test-helpers/pulls/19","html_url":"https://github.com/there4/slim-test-helpers/pull/19","diff_url":"https://github.com/there4/slim-test-helpers/pull/19.diff","patch_url":"https://github.com/there4/slim-test-helpers/pull/19.patch"},"body":"This commit will replace the __call magic with explicit method\r\ndeclarations for faking any of the supported HTTP methods.\r\n\r\nAs a API, it's preferable for the API to be discoverable. Having\r\nexplicit method declarations help IDEs to list the class methods when\r\nwriting tests."},"comment":{"url":"https://api.github.com/repos/there4/slim-test-helpers/issues/comments/125647231","html_url":"https://github.com/there4/slim-test-helpers/pull/19#issuecomment-125647231","issue_url":"https://api.github.com/repos/there4/slim-test-helpers/issues/19","id":125647231,"user":{"login":"craig-davis","id":9907304,"avatar_url":"https://avatars.githubusercontent.com/u/9907304?v=3","gravatar_id":"","url":"https://api.github.com/users/craig-davis","html_url":"https://github.com/craig-davis","followers_url":"https://api.github.com/users/craig-davis/followers","following_url":"https://api.github.com/users/craig-davis/following{/other_user}","gists_url":"https://api.github.com/users/craig-davis/gists{/gist_id}","starred_url":"https://api.github.com/users/craig-davis/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/craig-davis/subscriptions","organizations_url":"https://api.github.com/users/craig-davis/orgs","repos_url":"https://api.github.com/users/craig-davis/repos","events_url":"https://api.github.com/users/craig-davis/events{/privacy}","received_events_url":"https://api.github.com/users/craig-davis/received_events","type":"User","site_admin":false},"created_at":"2015-07-28T15:15:13Z","updated_at":"2015-07-28T15:15:13Z","body":"Hi @adbre - thank for sending these along. I'm a little torn on these PRs. My personal preference is always for less code. But, I can see where those programmers that choose to use an IDE might find this useful.\r\n\r\nI also prefer not to use PHP Doc style comments. I much prefer the literate code style, and don't like the clutter of PHP Doc comments that slowly drift out of sync. So, let me think about including these in the project. I have a feeling I need to allow the community of users of this dictate the style, and not my own preferences for small code and literate documentation.\r\n"}}