I am thinking it might be a good idea to make a new repo, with an eventual goal of putting each SDK into their own repos. It will allow the project to be published w/ composer (like maven).
Rod Simpson On Tue, Oct 28, 2014 at 8:34 AM, Dave <snoopd...@gmail.com> wrote: > Thanks Jason! > I don't know PHP so I'm not the best person to review this pull request, > but I did take a look. The project looks well-thought out and I did not > see any red flags like binary files or missing headers. > Here's the PR: https://github.com/apache/incubator-usergrid/pull/88 > Here are my questions for the team: > 1) We already have a PHP SDK, how is this one different or better than what > we have now? > 2) This is a big contribution, assuming it looks good to others: should we > simply pull it in to Usergrid master -- or do we need to do a code grant? > 3) The files have an ASL license header and also a copyright by > baas-platform.com. My understanding is that we do not require copyright > assignment, so that's fine, no? > Thoughts? > Any PHP experts out there want to weigh in on this contribution? Who wrote > our existing PHP SDK? > Thanks, > Dave > On Sun, Oct 26, 2014 at 1:55 AM, apps4u <g...@git.apache.org> wrote: >> Github user apps4u commented on the pull request: >> >> >> https://github.com/apache/incubator-usergrid/pull/88#issuecomment-60507519 >> >> Added some more api operation descriptors to version 1.0.1 of this SDK >> . It now has cleaner api and ResourceIteration as well. >> >>