)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":2394,"name":"Adam Spiers","email":"aspiers@suse.com","username":"adam.spiers"},"change_message_id":"451c5600780081029a2162fafcb5fed2ced239bd","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Bogdan Dobrelya \u003cbdobrelia@mirantis.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2015-10-30 14:01:04 +0100"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add a check for bashisms util"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Requires the devscripts and git packages installed"},{"line_number":10,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"1a26ad4f_ac6a927f","line":7,"updated":"2015-10-31 21:23:22.000000000","message":"This commit title is slightly unclear - I would suggest \"Add a script to run checkbashisms on codebase\" or similar.","commit_id":"fcaa4a31db91c0082aa4d63b43cfd8f51098241a"},{"author":{"_account_id":2394,"name":"Adam Spiers","email":"aspiers@suse.com","username":"adam.spiers"},"change_message_id":"451c5600780081029a2162fafcb5fed2ced239bd","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add a check for bashisms util"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Requires the devscripts and git packages installed"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Closes-bug: #1511721"},{"line_number":12,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"1a26ad4f_cc67ceb5","line":9,"updated":"2015-10-31 21:23:22.000000000","message":"A URL for devscripts would be helpful here.  Also, it is not necessary to install either of these dependencies via packages (they could be installed from sources), so I suggest dropping the word \u0027packages\u0027.","commit_id":"fcaa4a31db91c0082aa4d63b43cfd8f51098241a"}],"utils/syntax_check.sh":[{"author":{"_account_id":2394,"name":"Adam Spiers","email":"aspiers@suse.com","username":"adam.spiers"},"change_message_id":"451c5600780081029a2162fafcb5fed2ced239bd","unresolved":false,"context_lines":[{"line_number":24,"context_line":"POSIX_CHECKS\u003d1"},{"line_number":25,"context_line":"ALL\u003d1"},{"line_number":26,"context_line":"while [ $# -gt 0 ] ; do"},{"line_number":27,"context_line":"  key\u003d\"$1\""},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"  case $key in"},{"line_number":30,"context_line":"  -h|--help)"}],"source_content_type":"text/x-sh","patch_set":2,"id":"1a26ad4f_ec740a69","line":27,"updated":"2015-10-31 21:23:22.000000000","message":"The standard indentation level for this project is 4 columns; please could you adjust to that?","commit_id":"fcaa4a31db91c0082aa4d63b43cfd8f51098241a"},{"author":{"_account_id":2394,"name":"Adam Spiers","email":"aspiers@suse.com","username":"adam.spiers"},"change_message_id":"451c5600780081029a2162fafcb5fed2ced239bd","unresolved":false,"context_lines":[{"line_number":97,"context_line":"  return $exit_code"},{"line_number":98,"context_line":"}"},{"line_number":99,"context_line":""},{"line_number":100,"context_line":"results\u003d$(check_syntax)"},{"line_number":101,"context_line":"rc\u003d$?"},{"line_number":102,"context_line":"if [ $rc -eq 0 ] ; then"},{"line_number":103,"context_line":"  echo \"Syntax Test SUCCEEDED: no syntax errors found\""}],"source_content_type":"text/x-sh","patch_set":2,"id":"1a26ad4f_0c72a67b","line":100,"updated":"2015-10-31 21:23:22.000000000","message":"I think you can just do:\n\n    if results\u003d$(check_syntax); then","commit_id":"fcaa4a31db91c0082aa4d63b43cfd8f51098241a"}]}