{"action":"created","issue":{"url":"https://api.github.com/repos/ga4gh/schemas/issues/295","labels_url":"https://api.github.com/repos/ga4gh/schemas/issues/295/labels{/name}","comments_url":"https://api.github.com/repos/ga4gh/schemas/issues/295/comments","events_url":"https://api.github.com/repos/ga4gh/schemas/issues/295/events","html_url":"https://github.com/ga4gh/schemas/pull/295","id":71955328,"number":295,"title":"Add more CI","user":{"login":"dcolligan","id":7422120,"avatar_url":"https://avatars.githubusercontent.com/u/7422120?v=3","gravatar_id":"","url":"https://api.github.com/users/dcolligan","html_url":"https://github.com/dcolligan","followers_url":"https://api.github.com/users/dcolligan/followers","following_url":"https://api.github.com/users/dcolligan/following{/other_user}","gists_url":"https://api.github.com/users/dcolligan/gists{/gist_id}","starred_url":"https://api.github.com/users/dcolligan/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dcolligan/subscriptions","organizations_url":"https://api.github.com/users/dcolligan/orgs","repos_url":"https://api.github.com/users/dcolligan/repos","events_url":"https://api.github.com/users/dcolligan/events{/privacy}","received_events_url":"https://api.github.com/users/dcolligan/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":12,"created_at":"2015-04-29T18:44:54Z","updated_at":"2015-05-05T05:31:43Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/ga4gh/schemas/pulls/295","html_url":"https://github.com/ga4gh/schemas/pull/295","diff_url":"https://github.com/ga4gh/schemas/pull/295.diff","patch_url":"https://github.com/ga4gh/schemas/pull/295.patch"},"body":"- run mvn commands from python tests\r\n- throw error on mvn warnings\r\n- add test to compile and check _protocol_definitions.py file\r\n\r\nThis build will fail since #292 has not been resolved.  I will rebase this on top of master after that branch is merged in.  After this commit is merged into master, no branch should be merged into master that does not pass the tests."},"comment":{"url":"https://api.github.com/repos/ga4gh/schemas/issues/comments/98958180","html_url":"https://github.com/ga4gh/schemas/pull/295#issuecomment-98958180","issue_url":"https://api.github.com/repos/ga4gh/schemas/issues/295","id":98958180,"user":{"login":"diekhans","id":110053,"avatar_url":"https://avatars.githubusercontent.com/u/110053?v=3","gravatar_id":"","url":"https://api.github.com/users/diekhans","html_url":"https://github.com/diekhans","followers_url":"https://api.github.com/users/diekhans/followers","following_url":"https://api.github.com/users/diekhans/following{/other_user}","gists_url":"https://api.github.com/users/diekhans/gists{/gist_id}","starred_url":"https://api.github.com/users/diekhans/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/diekhans/subscriptions","organizations_url":"https://api.github.com/users/diekhans/orgs","repos_url":"https://api.github.com/users/diekhans/repos","events_url":"https://api.github.com/users/diekhans/events{/privacy}","received_events_url":"https://api.github.com/users/diekhans/received_events","type":"User","site_admin":false},"created_at":"2015-05-05T05:31:43Z","updated_at":"2015-05-05T05:31:43Z","body":"\nAt one point, I create \n  https://github.com/ga4gh/server/issues/81\n  generalize python code generation from schema #81\n\nassigned it to myself and never got back to it.\n\nThe thinking was that the class creator should be a program\nindependent of checking out a tree, etc.  Avro file input, class\noutput.  Worthy of submitting to Avro project on day.\n\nDanny Colligan <notifications@github.com> writes:\n> After conversations with @jeromekelleher I've decided to go with a new design.\n> This new code contains import_schemas.py which is only the functionality from\n> process_schemas.py that compiles the schemas into python objects, and none of\n> the code that does the writing of the _protocol_schemas.py file. Writing and\n> verifying _protocol_schemas.py will be the domain of the server repo code. This\n> design features minimal code duplication across both repos.\n> \n> —\n> Reply to this email directly or view it on GitHub.*\n> \n\n\n"}}