)]}'
{"searchlight_ui/api/rest/searchlight.py":[{"author":{"_account_id":10063,"name":"Steve McLellan","email":"steven.j.mclellan@gmail.com","username":"sjmc7"},"change_message_id":"c15e2065f8a7b36d9ce07c0f290e13f13588cb70","unresolved":false,"context_lines":[{"line_number":209,"context_line":""},{"line_number":210,"context_line":"def urljoin(*args):"},{"line_number":211,"context_line":"    \"\"\"Joins given arguments into a url. Trailing but not leading slashes are"},{"line_number":212,"context_line":""},{"line_number":213,"context_line":"    stripped for each argument."},{"line_number":214,"context_line":"    \"\"\""},{"line_number":215,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"3f1d235d_3416d34a","line":212,"updated":"2017-07-06 15:20:20.000000000","message":"This blank line is not necessary","commit_id":"139d3174804931c79def72484aa200c36bc60205"},{"author":{"_account_id":25254,"name":"Nguyen Van Trung","email":"trungnvfet@outlook.com","username":"trungnv"},"change_message_id":"b00219c77d32fa9a771263c0f00a1f76799aabee","unresolved":false,"context_lines":[{"line_number":209,"context_line":""},{"line_number":210,"context_line":"def urljoin(*args):"},{"line_number":211,"context_line":"    \"\"\"Joins given arguments into a url. Trailing but not leading slashes are"},{"line_number":212,"context_line":""},{"line_number":213,"context_line":"    stripped for each argument."},{"line_number":214,"context_line":"    \"\"\""},{"line_number":215,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"3f1d235d_9c9db7bd","line":212,"in_reply_to":"3f1d235d_3416d34a","updated":"2017-07-07 06:16:31.000000000","message":"Done","commit_id":"139d3174804931c79def72484aa200c36bc60205"},{"author":{"_account_id":23630,"name":"Tovin Seven","email":"tovin07@gmail.com","username":"tovin07"},"change_message_id":"ab4c0754c315932541c59857752b56d19fed73f8","unresolved":false,"context_lines":[{"line_number":207,"context_line":"    return urljoin(searchlight_url, \u0027v1\u0027)"},{"line_number":208,"context_line":""},{"line_number":209,"context_line":""},{"line_number":210,"context_line":"def urljoin(*args):"},{"line_number":211,"context_line":"    \"\"\"Joins given arguments into a url."},{"line_number":212,"context_line":""},{"line_number":213,"context_line":"    Trailing but not leading slashes are stripped for each argument."}],"source_content_type":"text/x-python","patch_set":5,"id":"3f1d235d_2d752199","line":210,"range":{"start_line":210,"start_character":4,"end_line":210,"end_character":11},"updated":"2017-07-10 07:00:19.000000000","message":"if this function is used in #207 only, a new function for this will not make sense at all.\njust use string concatenate for simple.","commit_id":"eb1dee7a6137d5fa8e8dfd1ceebfca0c0b8da88a"},{"author":{"_account_id":23630,"name":"Tovin Seven","email":"tovin07@gmail.com","username":"tovin07"},"change_message_id":"ab4c0754c315932541c59857752b56d19fed73f8","unresolved":false,"context_lines":[{"line_number":213,"context_line":"    Trailing but not leading slashes are stripped for each argument."},{"line_number":214,"context_line":"    \"\"\""},{"line_number":215,"context_line":""},{"line_number":216,"context_line":"    return \"/\".join(map(lambda x: x.rstrip(\u0027/\u0027), args))"}],"source_content_type":"text/x-python","patch_set":5,"id":"3f1d235d_ed20499e","line":216,"range":{"start_line":216,"start_character":36,"end_line":216,"end_character":42},"updated":"2017-07-10 07:00:19.000000000","message":".strip should be used too (for \"/v1\")","commit_id":"eb1dee7a6137d5fa8e8dfd1ceebfca0c0b8da88a"},{"author":{"_account_id":23630,"name":"Tovin Seven","email":"tovin07@gmail.com","username":"tovin07"},"change_message_id":"ab4c0754c315932541c59857752b56d19fed73f8","unresolved":false,"context_lines":[{"line_number":213,"context_line":"    Trailing but not leading slashes are stripped for each argument."},{"line_number":214,"context_line":"    \"\"\""},{"line_number":215,"context_line":""},{"line_number":216,"context_line":"    return \"/\".join(map(lambda x: x.rstrip(\u0027/\u0027), args))"}],"source_content_type":"text/x-python","patch_set":5,"id":"3f1d235d_f2b024e0","line":216,"range":{"start_line":216,"start_character":20,"end_line":216,"end_character":54},"updated":"2017-07-10 07:00:19.000000000","message":"list comprehension is better readable for this case.\n\njust don\u0027t blindly copy from this https://stackoverflow.com/a/11326230","commit_id":"eb1dee7a6137d5fa8e8dfd1ceebfca0c0b8da88a"}],"searchlight_ui/tests/urls.py":[{"author":{"_account_id":23630,"name":"Tovin Seven","email":"tovin07@gmail.com","username":"tovin07"},"change_message_id":"ab4c0754c315932541c59857752b56d19fed73f8","unresolved":false,"context_lines":[{"line_number":14,"context_line":"from django.conf import urls"},{"line_number":15,"context_line":"import openstack_dashboard.urls"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"urlpatterns \u003d ["},{"line_number":18,"context_line":"    \u0027\u0027,"},{"line_number":19,"context_line":"    urls.url(r\u0027\u0027, urls.include(openstack_dashboard.urls))"},{"line_number":20,"context_line":"]"}],"source_content_type":"text/x-python","patch_set":5,"id":"3f1d235d_4db4dd35","line":17,"range":{"start_line":17,"start_character":13,"end_line":17,"end_character":15},"updated":"2017-07-10 07:00:19.000000000","message":"please drop some words in commit message for this deprecation","commit_id":"eb1dee7a6137d5fa8e8dfd1ceebfca0c0b8da88a"},{"author":{"_account_id":25254,"name":"Nguyen Van Trung","email":"trungnvfet@outlook.com","username":"trungnv"},"change_message_id":"38b7a64003f570efc0edcaecbebdae61a1e70d05","unresolved":false,"context_lines":[{"line_number":14,"context_line":"from django.conf import urls"},{"line_number":15,"context_line":"import openstack_dashboard.urls"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"urlpatterns \u003d ["},{"line_number":18,"context_line":"    \u0027\u0027,"},{"line_number":19,"context_line":"    urls.url(r\u0027\u0027, urls.include(openstack_dashboard.urls))"},{"line_number":20,"context_line":"]"}],"source_content_type":"text/x-python","patch_set":5,"id":"3f1d235d_4d9a9d6e","line":17,"range":{"start_line":17,"start_character":13,"end_line":17,"end_character":15},"in_reply_to":"3f1d235d_4db4dd35","updated":"2017-07-10 07:13:02.000000000","message":"Yep. This change just help to pass jenkin at the moment. thus I suggested to new PS then rebase on it.","commit_id":"eb1dee7a6137d5fa8e8dfd1ceebfca0c0b8da88a"}]}