)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"d759cb76607c4cb2692dc62d93acd415cb7cda60","unresolved":true,"context_lines":[{"line_number":4,"context_line":"Commit:     Jorhson Deng \u003cdengzhaosen@inspur.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2021-06-24 02:21:12 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Ussuri+ is python3 only and update python to python3"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"remove the redundant shebang and the colorizer.py which"},{"line_number":10,"context_line":"hasn\u0027t been used since https://review.opendev.org/c/openstack/python-cinderclient/+/502120"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"6fde93bf_2fd85936","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":52},"updated":"2021-08-25 20:47:20.000000000","message":"I think you can be more specific here about what this patch is doing, something like \"Update some tool scripts\" and then say what your updates are in the body of the commit message.","commit_id":"6b823919262e3076233bba4fdcf3d90dfc2eb2dd"}],"tools/lintstack.py":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"d759cb76607c4cb2692dc62d93acd415cb7cda60","unresolved":true,"context_lines":[{"line_number":1,"context_line":"#!/usr/bin/env python"},{"line_number":2,"context_line":"# Copyright (c) 2013, AT\u0026T Labs, Yun Mao \u003cyunmao@gmail.com\u003e"},{"line_number":3,"context_line":"# All Rights Reserved."},{"line_number":4,"context_line":"#"}],"source_content_type":"text/x-python","patch_set":2,"id":"108fadc3_4350f5db","side":"PARENT","line":1,"updated":"2021-08-25 20:47:20.000000000","message":"I think you can\u0027t simply remove this because of the way this file is called from linstack.sh","commit_id":"c559f090ff11904455f31ce2cb595e8f246f06e9"}],"tools/lintstack.sh":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"d759cb76607c4cb2692dc62d93acd415cb7cda60","unresolved":true,"context_lines":[{"line_number":50,"context_line":"fi"},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"# First generate tools/pylint_exceptions from HEAD~1"},{"line_number":53,"context_line":"$TOOLS_DIR/lintstack.head.py generate"},{"line_number":54,"context_line":"# Then use that as a reference to compare against HEAD"},{"line_number":55,"context_line":"git checkout $GITHEAD"},{"line_number":56,"context_line":"$TOOLS_DIR/lintstack.head.py"}],"source_content_type":"text/x-sh","patch_set":2,"id":"6464a4c6_619b35d4","line":53,"range":{"start_line":53,"start_character":0,"end_line":53,"end_character":28},"updated":"2021-08-25 20:47:20.000000000","message":"when lintstack.py is called like this and it no longer contains a shebang, bash thinks it\u0027s a bash script and will start barfing on python syntax that it doesn\u0027t understand","commit_id":"6b823919262e3076233bba4fdcf3d90dfc2eb2dd"},{"author":{"_account_id":31827,"name":"Jorhson Deng","email":"jorhson_deng@163.com","username":"dengzhaosen"},"change_message_id":"4ba1035c73c78c9348846d5f6fea1d26deb36e8f","unresolved":true,"context_lines":[{"line_number":50,"context_line":"fi"},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"# First generate tools/pylint_exceptions from HEAD~1"},{"line_number":53,"context_line":"$TOOLS_DIR/lintstack.head.py generate"},{"line_number":54,"context_line":"# Then use that as a reference to compare against HEAD"},{"line_number":55,"context_line":"git checkout $GITHEAD"},{"line_number":56,"context_line":"$TOOLS_DIR/lintstack.head.py"}],"source_content_type":"text/x-sh","patch_set":2,"id":"80b68802_e86041f1","line":53,"range":{"start_line":53,"start_character":0,"end_line":53,"end_character":28},"in_reply_to":"6464a4c6_619b35d4","updated":"2021-08-26 01:36:44.000000000","message":"there is no need use the shebang in python3 because it will use the python to excute the file in default.","commit_id":"6b823919262e3076233bba4fdcf3d90dfc2eb2dd"},{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"bf4b5e3aaeca3ab6bb71c4e9f1bd32fdddfee2ee","unresolved":true,"context_lines":[{"line_number":50,"context_line":"fi"},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"# First generate tools/pylint_exceptions from HEAD~1"},{"line_number":53,"context_line":"$TOOLS_DIR/lintstack.head.py generate"},{"line_number":54,"context_line":"# Then use that as a reference to compare against HEAD"},{"line_number":55,"context_line":"git checkout $GITHEAD"},{"line_number":56,"context_line":"$TOOLS_DIR/lintstack.head.py"}],"source_content_type":"text/x-sh","patch_set":2,"id":"4e569d24_84ae8102","line":53,"range":{"start_line":53,"start_character":0,"end_line":53,"end_character":28},"in_reply_to":"80b68802_e86041f1","updated":"2021-08-30 14:24:24.000000000","message":"That\u0027s not always the case though. The shebang should be kept.","commit_id":"6b823919262e3076233bba4fdcf3d90dfc2eb2dd"}]}