{"action":"created","issue":{"url":"https://api.github.com/repos/mezuro/kalibro_processor/issues/106","labels_url":"https://api.github.com/repos/mezuro/kalibro_processor/issues/106/labels{/name}","comments_url":"https://api.github.com/repos/mezuro/kalibro_processor/issues/106/comments","events_url":"https://api.github.com/repos/mezuro/kalibro_processor/issues/106/events","html_url":"https://github.com/mezuro/kalibro_processor/pull/106","id":88781256,"number":106,"title":"Adding radon as python metric collector","user":{"login":"MaxAlmeida","id":5316537,"avatar_url":"https://avatars.githubusercontent.com/u/5316537?v=3","gravatar_id":"","url":"https://api.github.com/users/MaxAlmeida","html_url":"https://github.com/MaxAlmeida","followers_url":"https://api.github.com/users/MaxAlmeida/followers","following_url":"https://api.github.com/users/MaxAlmeida/following{/other_user}","gists_url":"https://api.github.com/users/MaxAlmeida/gists{/gist_id}","starred_url":"https://api.github.com/users/MaxAlmeida/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/MaxAlmeida/subscriptions","organizations_url":"https://api.github.com/users/MaxAlmeida/orgs","repos_url":"https://api.github.com/users/MaxAlmeida/repos","events_url":"https://api.github.com/users/MaxAlmeida/events{/privacy}","received_events_url":"https://api.github.com/users/MaxAlmeida/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":7,"created_at":"2015-06-16T16:51:36Z","updated_at":"2015-07-03T12:03:06Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/mezuro/kalibro_processor/pulls/106","html_url":"https://github.com/mezuro/kalibro_processor/pull/106","diff_url":"https://github.com/mezuro/kalibro_processor/pull/106.diff","patch_url":"https://github.com/mezuro/kalibro_processor/pull/106.patch"},"body":"It's solve the issue #83 \r\n\r\nSigned-off-by: MaxAlmeida <llewxam100@hotmail.com>\r\nSigned-off-by: LucasLeite <lucas.srl08@gmail.com>\r\nSigned-off-by: Parley Martins <parleypachecomartins@gmail.com>"},"comment":{"url":"https://api.github.com/repos/mezuro/kalibro_processor/issues/comments/118330018","html_url":"https://github.com/mezuro/kalibro_processor/pull/106#issuecomment-118330018","issue_url":"https://api.github.com/repos/mezuro/kalibro_processor/issues/106","id":118330018,"user":{"login":"rafamanzo","id":288884,"avatar_url":"https://avatars.githubusercontent.com/u/288884?v=3","gravatar_id":"","url":"https://api.github.com/users/rafamanzo","html_url":"https://github.com/rafamanzo","followers_url":"https://api.github.com/users/rafamanzo/followers","following_url":"https://api.github.com/users/rafamanzo/following{/other_user}","gists_url":"https://api.github.com/users/rafamanzo/gists{/gist_id}","starred_url":"https://api.github.com/users/rafamanzo/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rafamanzo/subscriptions","organizations_url":"https://api.github.com/users/rafamanzo/orgs","repos_url":"https://api.github.com/users/rafamanzo/repos","events_url":"https://api.github.com/users/rafamanzo/events{/privacy}","received_events_url":"https://api.github.com/users/rafamanzo/received_events","type":"User","site_admin":false},"created_at":"2015-07-03T12:03:06Z","updated_at":"2015-07-03T12:03:06Z","body":"Just finished another review round. Good job with the fixes. See the remaning issues below:\r\n\r\n- [ ] `./spec/lib/metric_collector/native/radon/parser_spec.rb:14` still fails if you don't have radon installed\r\n- [ ] Handle radon error on parsers (more details below).\r\n\r\nI can confirm that the acceptance test fails if you install radon through Python 3. Specifically this happens for your sample project on the file `api/busline.py` which synthax is invalid under Python 3. This makes me be sure that you must handle on your parsers the case where radon returns a error like the example given before:\r\n\r\n```ruby\r\n[\"error\", \"invalid syntax (<unknown>, line 75)\"]\r\n```\r\n\r\nSigned off by: @danielkza "}}