{"action":"created","issue":{"url":"https://api.github.com/repos/vaffel/sql-to-graphql/issues/36","labels_url":"https://api.github.com/repos/vaffel/sql-to-graphql/issues/36/labels{/name}","comments_url":"https://api.github.com/repos/vaffel/sql-to-graphql/issues/36/comments","events_url":"https://api.github.com/repos/vaffel/sql-to-graphql/issues/36/events","html_url":"https://github.com/vaffel/sql-to-graphql/pull/36","id":110290768,"number":36,"title":"first cut at sqlite support","user":{"login":"jtomson","id":380187,"avatar_url":"https://avatars.githubusercontent.com/u/380187?v=3","gravatar_id":"","url":"https://api.github.com/users/jtomson","html_url":"https://github.com/jtomson","followers_url":"https://api.github.com/users/jtomson/followers","following_url":"https://api.github.com/users/jtomson/following{/other_user}","gists_url":"https://api.github.com/users/jtomson/gists{/gist_id}","starred_url":"https://api.github.com/users/jtomson/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jtomson/subscriptions","organizations_url":"https://api.github.com/users/jtomson/orgs","repos_url":"https://api.github.com/users/jtomson/repos","events_url":"https://api.github.com/users/jtomson/events{/privacy}","received_events_url":"https://api.github.com/users/jtomson/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/vaffel/sql-to-graphql/labels/enhancement","name":"enhancement","color":"84b6eb"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":3,"created_at":"2015-10-07T18:44:38Z","updated_at":"2015-10-13T16:01:17Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/vaffel/sql-to-graphql/pulls/36","html_url":"https://github.com/vaffel/sql-to-graphql/pull/36","diff_url":"https://github.com/vaffel/sql-to-graphql/pull/36.diff","patch_url":"https://github.com/vaffel/sql-to-graphql/pull/36.patch"},"body":"Very cool project! This PR addresses issue #17, adding basic sqlite support"},"comment":{"url":"https://api.github.com/repos/vaffel/sql-to-graphql/issues/comments/147760009","html_url":"https://github.com/vaffel/sql-to-graphql/pull/36#issuecomment-147760009","issue_url":"https://api.github.com/repos/vaffel/sql-to-graphql/issues/36","id":147760009,"user":{"login":"rexxars","id":48200,"avatar_url":"https://avatars.githubusercontent.com/u/48200?v=3","gravatar_id":"","url":"https://api.github.com/users/rexxars","html_url":"https://github.com/rexxars","followers_url":"https://api.github.com/users/rexxars/followers","following_url":"https://api.github.com/users/rexxars/following{/other_user}","gists_url":"https://api.github.com/users/rexxars/gists{/gist_id}","starred_url":"https://api.github.com/users/rexxars/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rexxars/subscriptions","organizations_url":"https://api.github.com/users/rexxars/orgs","repos_url":"https://api.github.com/users/rexxars/repos","events_url":"https://api.github.com/users/rexxars/events{/privacy}","received_events_url":"https://api.github.com/users/rexxars/received_events","type":"User","site_admin":false},"created_at":"2015-10-13T16:01:17Z","updated_at":"2015-10-13T16:01:17Z","body":"This is great - it does have a few flaws though. I tried it on the chinook (https://chinookdatabase.codeplex.com/) database, and it has a hard time recognizing some types:\r\n\r\n- Types are not lowercased, so `INTEGER` is not recognized\r\n- There is apparently an `nvarchar` type that we should be able to map directly to a string, but it has a size specifier that needs to be removed (`nvarchar(36)`, for instance)\r\n\r\nThere is also several lint errors which would be nice to get rid of. If you're still keen on getting this merged, could you download the chinook example database and try to get this running on that?\r\n\r\nMake sure to run `npm test` to run the linter once you've got it working :-)\r\n"}}