Closed elof closed 8 years ago
LGTM. Do you want to go ahead and delete query/README.md so that it isn't duplicated in two places? My concern is that the two might accidentally diverge and cause confusion.
Also, just FYI, the Travis build failure seems like it's probably this issue: https://github.com/travis-ci/travis-ci/issues/5669 and therefore should be safe to ignore for now; looks like they're hoping to have a fix soon.
@geeber I think that's a good idea. I'll remove the second readme.
@Geeber let me know if you think this is ready to merge meow :)
Yep LGTM.
Took the info from the README in the query file and reformated it to fit the style of the main readme. Now adding queries to the SDK is not long under TODO :D