)]}'
{"trove/tests/util/users.py":[{"author":{"_account_id":6162,"name":"Dan Nguyen","email":"dan.nguyens.mail@gmail.com","username":"daniel-a-nguyen"},"change_message_id":"c3cc2209a766a3339ed1de0410250e2ebd7a26ca","unresolved":false,"context_lines":[{"line_number":56,"context_line":""},{"line_number":57,"context_line":"    def __init__(self, auth_user\u003dNone, auth_key\u003dNone, services\u003dNone,"},{"line_number":58,"context_line":"                 tenant\u003dNone, tenant_id\u003dNone, requirements\u003dNone):"},{"line_number":59,"context_line":"        \"\"\"Creates info on a user.\"\"\""},{"line_number":60,"context_line":"        self.auth_user \u003d auth_user"},{"line_number":61,"context_line":"        self.auth_key \u003d auth_key"},{"line_number":62,"context_line":"        self.tenant \u003d tenant"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAUH%2F%2B66I%3D","line":59,"updated":"2014-01-20 18:34:59.000000000","message":"The __init__ method still can take auth_key, services, and tenant.\nAlthough it\u0027s arguable that we should show the auth_key it might make sense to show tenant, and services if they are present.\n\nConsider checking to see if tenant and services are present and adding them to __str__.","commit_id":"67e69bcda2eabb74cf2489dfe9f57e268744f74e"},{"author":{"_account_id":7806,"name":"Sushil Kumar","email":"skm.net@gmail.com","username":"sushilkm"},"change_message_id":"6f8400c8ae14a6fe7ef20c2d3722c7a67e8177ce","unresolved":false,"context_lines":[{"line_number":56,"context_line":""},{"line_number":57,"context_line":"    def __init__(self, auth_user\u003dNone, auth_key\u003dNone, services\u003dNone,"},{"line_number":58,"context_line":"                 tenant\u003dNone, tenant_id\u003dNone, requirements\u003dNone):"},{"line_number":59,"context_line":"        \"\"\"Creates info on a user.\"\"\""},{"line_number":60,"context_line":"        self.auth_user \u003d auth_user"},{"line_number":61,"context_line":"        self.auth_key \u003d auth_key"},{"line_number":62,"context_line":"        self.tenant \u003d tenant"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAUH%2F%2BTts%3D","line":59,"in_reply_to":"AAAAUH%2F%2B66I%3D","updated":"2014-01-28 05:49:10.000000000","message":"I would say that, what you are suggesting is out of scope of current bug. It could be considered separately\n\nThis patchset is aimed to correct __str__ method which has got statements after a return call, which are no use.","commit_id":"67e69bcda2eabb74cf2489dfe9f57e268744f74e"},{"author":{"_account_id":7806,"name":"Sushil Kumar","email":"skm.net@gmail.com","username":"sushilkm"},"change_message_id":"51c9be7e2a68a7f81ae431bee227c793689fbcb3","unresolved":false,"context_lines":[{"line_number":56,"context_line":""},{"line_number":57,"context_line":"    def __init__(self, auth_user\u003dNone, auth_key\u003dNone, services\u003dNone,"},{"line_number":58,"context_line":"                 tenant\u003dNone, tenant_id\u003dNone, requirements\u003dNone):"},{"line_number":59,"context_line":"        \"\"\"Creates info on a user.\"\"\""},{"line_number":60,"context_line":"        self.auth_user \u003d auth_user"},{"line_number":61,"context_line":"        self.auth_key \u003d auth_key"},{"line_number":62,"context_line":"        self.tenant \u003d tenant"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAXX%2F%2FwCo%3D","line":59,"in_reply_to":"AAAAUH%2F%2BA2s%3D","updated":"2014-04-28 08:55:18.000000000","message":"1. I have written a change in file related to tests.\n2. What I get from the review is that we need to remove auth_key from the code.\nJust needed to confirm if I need to do the same in the current patch-set, because it is entirely something which is un-related to current bug, which was only about that there exists some code after return statement in \"__str__ method.","commit_id":"67e69bcda2eabb74cf2489dfe9f57e268744f74e"},{"author":{"_account_id":739,"name":"Michael Basnight","email":"mbasnight@gmail.com","username":"hubcap"},"change_message_id":"8d40fbd6f76d226195ade0f105e7d8fed8decf1d","unresolved":false,"context_lines":[{"line_number":56,"context_line":""},{"line_number":57,"context_line":"    def __init__(self, auth_user\u003dNone, auth_key\u003dNone, services\u003dNone,"},{"line_number":58,"context_line":"                 tenant\u003dNone, tenant_id\u003dNone, requirements\u003dNone):"},{"line_number":59,"context_line":"        \"\"\"Creates info on a user.\"\"\""},{"line_number":60,"context_line":"        self.auth_user \u003d auth_user"},{"line_number":61,"context_line":"        self.auth_key \u003d auth_key"},{"line_number":62,"context_line":"        self.tenant \u003d tenant"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAUH%2F%2BA2s%3D","line":59,"in_reply_to":"AAAAUH%2F%2BTts%3D","updated":"2014-01-30 23:55:11.000000000","message":"Thats silly. do them both here.","commit_id":"67e69bcda2eabb74cf2489dfe9f57e268744f74e"}]}