)]}'
{"tests/test_v3_auth.py":[{"author":{"_account_id":6456,"name":"Sahdev Zala","email":"spzala@us.ibm.com","username":"spzala"},"change_message_id":"0774f97a833d21d2a0023c78c9cd9d41f2547a45","unresolved":false,"context_lines":[{"line_number":22,"context_line":"from keystone import config"},{"line_number":23,"context_line":"from keystone import exception"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"import test_v3"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"CONF \u003d config.CONF"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAPH%2F%2FvmI%3D","line":25,"updated":"2013-05-23 23:42:52.000000000","message":"I just went for -1 .. thought if you don\u0027t get chance to look at the comment and it gets enough + then it may go in the build unnoticed.","commit_id":"c08461f4352175ed7c00837ca979949b5993dc80"},{"author":{"_account_id":6456,"name":"Sahdev Zala","email":"spzala@us.ibm.com","username":"spzala"},"change_message_id":"3f735475c076e32e794aa025eb86b5f6198f7e61","unresolved":false,"context_lines":[{"line_number":22,"context_line":"from keystone import config"},{"line_number":23,"context_line":"from keystone import exception"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"import test_v3"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"CONF \u003d config.CONF"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAPH%2F%2FvuM%3D","line":25,"updated":"2013-05-23 23:03:56.000000000","message":"Shouldn\u0027t this go after nose.exc? \nI could be wrong, it\u0027s confusing specially with long statements at few places :-) but getting it.","commit_id":"c08461f4352175ed7c00837ca979949b5993dc80"},{"author":{"_account_id":6456,"name":"Sahdev Zala","email":"spzala@us.ibm.com","username":"spzala"},"change_message_id":"e6de35a3381cb617b96f6f3fa1a673186f2500d6","unresolved":false,"context_lines":[{"line_number":22,"context_line":"from keystone import config"},{"line_number":23,"context_line":"from keystone import exception"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"import test_v3"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"CONF \u003d config.CONF"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAPH%2F%2FtZY%3D","line":25,"in_reply_to":"AAAAPH%2F%2Ftaw%3D","updated":"2013-05-24 19:38:20.000000000","message":"Ah, cool. Thanks for the detail explanation, Dolph! This is good to know.","commit_id":"c08461f4352175ed7c00837ca979949b5993dc80"},{"author":{"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},"change_message_id":"736f1e23d63dab48c48f23b47828b446abbb5328","unresolved":false,"context_lines":[{"line_number":22,"context_line":"from keystone import config"},{"line_number":23,"context_line":"from keystone import exception"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"import test_v3"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"CONF \u003d config.CONF"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAPH%2F%2Ftaw%3D","line":25,"in_reply_to":"AAAAPH%2F%2FvmI%3D","updated":"2013-05-24 19:25:39.000000000","message":"they\u0027re are group correctly here, according to...\n\n- json/uuid are stdlib imports\n- nose is an external package import\n- keystone is all local imports\n- test_v3 is a relative import\n\nrelative imports are actually a no-no, and checked by flake8 H304, which we\u0027re still ignoring (and have to) because there\u0027s no package to contain them today. termie added an __init__ to tests/ once, but it got reverted because people had environment issues with it.","commit_id":"c08461f4352175ed7c00837ca979949b5993dc80"}]}