)]}'
{"zunclient/v1/containers.py":[{"author":{"_account_id":11536,"name":"hongbin","email":"hongbin034@gmail.com","username":"hongbin"},"change_message_id":"801bd67c0213682bcc4cff4a981f442858f9f756","unresolved":false,"context_lines":[{"line_number":96,"context_line":"        new \u003d {}"},{"line_number":97,"context_line":"        for (key, value) in kwargs.items():"},{"line_number":98,"context_line":"            if key in CREATION_ATTRIBUTES:"},{"line_number":99,"context_line":"                if value:"},{"line_number":100,"context_line":"                    new[key] \u003d value"},{"line_number":101,"context_line":"            else:"},{"line_number":102,"context_line":"                raise exceptions.InvalidAttribute("},{"line_number":103,"context_line":"                    \"Key must be in %s\" % \u0027,\u0027.join(CREATION_ATTRIBUTES))"}],"source_content_type":"text/x-python","patch_set":1,"id":"1a430d35_5d004766","line":100,"range":{"start_line":99,"start_character":0,"end_line":100,"end_character":36},"updated":"2017-02-08 22:51:41.000000000","message":"@Kevin,\n\nWe explicitly do the null checking in shell: https://github.com/openstack/python-zunclient/blob/master/zunclient/v1/containers_shell.py#L124 . I think it is better to do the check in there for consistency.","commit_id":"2bc79cca4e8483c4a3a694276b60bc464ce0a93f"}]}