)]}'
{"monascaclient/shell.py":[{"author":{"_account_id":24580,"name":"James Gu","email":"james.gu@microsoft.com","username":"jamesgu"},"change_message_id":"a7853d049d8063261a2b97558b6a5ca6ac8db7c6","unresolved":false,"context_lines":[{"line_number":341,"context_line":"                args.os_auth_url +\u003d \u0027/\u0027"},{"line_number":342,"context_line":"            args.os_auth_url +\u003d args.os_auth_version"},{"line_number":343,"context_line":""},{"line_number":344,"context_line":"        if args.os_auth_url and \u0027v2.0\u0027 in args.os_auth_url:"},{"line_number":345,"context_line":"            args.os_auth_url \u003d string.replace(args.os_auth_url, \u0027v2.0\u0027, \u0027v3\u0027)"},{"line_number":346,"context_line":""},{"line_number":347,"context_line":"        kwargs \u003d {"}],"source_content_type":"text/x-python","patch_set":4,"id":"dfeb2761_cedf6238","line":344,"range":{"start_line":344,"start_character":0,"end_line":344,"end_character":59},"updated":"2017-04-04 16:28:08.000000000","message":"hhmmm interesting two lines indeed. This commit on the other hand achieved what it wanted to do, allows to specify the version separately from the auth url. What I am wondering is the use case that requires the separation of the auth url vs version number.","commit_id":"26c73d244907d11b5e6cc793d96012f6871b7356"},{"author":{"_account_id":16222,"name":"witek","email":"witold.bedyk@suse.com","username":"witek"},"change_message_id":"5c60e1eaf5c6647f5e0c78b7207b62e04c0f6184","unresolved":false,"context_lines":[{"line_number":341,"context_line":"                args.os_auth_url +\u003d \u0027/\u0027"},{"line_number":342,"context_line":"            args.os_auth_url +\u003d args.os_auth_version"},{"line_number":343,"context_line":""},{"line_number":344,"context_line":"        if args.os_auth_url and \u0027v2.0\u0027 in args.os_auth_url:"},{"line_number":345,"context_line":"            args.os_auth_url \u003d string.replace(args.os_auth_url, \u0027v2.0\u0027, \u0027v3\u0027)"},{"line_number":346,"context_line":""},{"line_number":347,"context_line":"        kwargs \u003d {"},{"line_number":348,"context_line":"            \u0027username\u0027: args.os_username,"}],"source_content_type":"text/x-python","patch_set":4,"id":"dfeb2761_fdbe62b2","line":345,"range":{"start_line":344,"start_character":8,"end_line":345,"end_character":77},"updated":"2017-04-04 15:25:05.000000000","message":"These lines will overwrite your setting anyway.","commit_id":"26c73d244907d11b5e6cc793d96012f6871b7356"},{"author":{"_account_id":16222,"name":"witek","email":"witold.bedyk@suse.com","username":"witek"},"change_message_id":"2c527305fe60960c11b7cf325a6896b8722a4d7e","unresolved":false,"context_lines":[{"line_number":334,"context_line":"                                       \" either --os-auth-url or via\""},{"line_number":335,"context_line":"                                       \" env[OS_AUTH_URL]\")"},{"line_number":336,"context_line":""},{"line_number":337,"context_line":"        authvarspresent \u003d args.os_auth_url and args.os_auth_version"},{"line_number":338,"context_line":"        versioned \u003d \u0027v2.0\u0027 in args.os_auth_url or \u0027v3\u0027 in args.os_auth_url"},{"line_number":339,"context_line":"        if authvarspresent and not versioned:"},{"line_number":340,"context_line":"            if not args.os_auth_url.endswith(\u0027/\u0027):"}],"source_content_type":"text/x-python","patch_set":5,"id":"bff0334d_1b68beb6","line":337,"range":{"start_line":337,"start_character":8,"end_line":337,"end_character":23},"updated":"2017-04-11 10:58:42.000000000","message":"Please separate words by underscores, \u0027auth_vars_present\u0027.","commit_id":"9d2594401ab064e0c103f2ad49f1a39a70b63819"},{"author":{"_account_id":16222,"name":"witek","email":"witold.bedyk@suse.com","username":"witek"},"change_message_id":"2c527305fe60960c11b7cf325a6896b8722a4d7e","unresolved":false,"context_lines":[{"line_number":337,"context_line":"        authvarspresent \u003d args.os_auth_url and args.os_auth_version"},{"line_number":338,"context_line":"        versioned \u003d \u0027v2.0\u0027 in args.os_auth_url or \u0027v3\u0027 in args.os_auth_url"},{"line_number":339,"context_line":"        if authvarspresent and not versioned:"},{"line_number":340,"context_line":"            if not args.os_auth_url.endswith(\u0027/\u0027):"},{"line_number":341,"context_line":"                args.os_auth_url +\u003d \u0027/\u0027"},{"line_number":342,"context_line":"            args.os_auth_url +\u003d args.os_auth_version"},{"line_number":343,"context_line":""},{"line_number":344,"context_line":"        if args.os_auth_url and \u0027v2.0\u0027 in args.os_auth_url:"},{"line_number":345,"context_line":"            args.os_auth_url \u003d string.replace(args.os_auth_url, \u0027v2.0\u0027, \u0027v3\u0027)"}],"source_content_type":"text/x-python","patch_set":5,"id":"bff0334d_f46087d1","line":342,"range":{"start_line":340,"start_character":12,"end_line":342,"end_character":52},"updated":"2017-04-11 10:58:42.000000000","message":"You could use urlparse.urljoin instead.","commit_id":"9d2594401ab064e0c103f2ad49f1a39a70b63819"},{"author":{"_account_id":16222,"name":"witek","email":"witold.bedyk@suse.com","username":"witek"},"change_message_id":"415e7f74fdb9bb3224d3100e5aea584f525f9d79","unresolved":false,"context_lines":[{"line_number":23,"context_line":"import logging"},{"line_number":24,"context_line":"import string"},{"line_number":25,"context_line":"import sys"},{"line_number":26,"context_line":"from urlparse import urljoin"},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"import monascaclient"},{"line_number":29,"context_line":"from monascaclient import client as monasca_client"}],"source_content_type":"text/x-python","patch_set":6,"id":"bff0334d_7cfc593d","line":26,"range":{"start_line":26,"start_character":0,"end_line":26,"end_character":28},"updated":"2017-04-11 14:36:40.000000000","message":"I\u0027m sorry for misleading comment. Please use six.moves.urllib.parse.urljoin instead.","commit_id":"7a8355bb2f8b9c1826cd3a423ba2ca7f71f9ab2a"}]}