)]}'
{"jenkins/scripts/common_translation_update.sh":[{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"b30686e632770b5ce5e86cce228a0a3924d66901","unresolved":false,"context_lines":[{"line_number":207,"context_line":"        set +e"},{"line_number":208,"context_line":"        # We cannot rely on the default branch in .gitreview being"},{"line_number":209,"context_line":"        # correct so we are very explicit here."},{"line_number":210,"context_line":"        OUTPUT\u003d$(git review -t zanata/translations $branch)"},{"line_number":211,"context_line":"        RET\u003d$?"},{"line_number":212,"context_line":"        [[ \"$RET\" -eq \"0\" || \"$OUTPUT\" \u003d~ \"No changes between prior commit\" ]]"},{"line_number":213,"context_line":"        SUCCESS\u003d$?"}],"source_content_type":"text/x-sh","patch_set":1,"id":"1a26ad4f_960b1d8e","line":210,"updated":"2015-11-02 01:42:21.000000000","message":"why not locals?","commit_id":"4f249a98da0c620f57f558039e5cb49d9c3dcd94"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"b30686e632770b5ce5e86cce228a0a3924d66901","unresolved":false,"context_lines":[{"line_number":210,"context_line":"        OUTPUT\u003d$(git review -t zanata/translations $branch)"},{"line_number":211,"context_line":"        RET\u003d$?"},{"line_number":212,"context_line":"        [[ \"$RET\" -eq \"0\" || \"$OUTPUT\" \u003d~ \"No changes between prior commit\" ]]"},{"line_number":213,"context_line":"        SUCCESS\u003d$?"},{"line_number":214,"context_line":"        set -e"},{"line_number":215,"context_line":"    fi"},{"line_number":216,"context_line":"}"}],"source_content_type":"text/x-sh","patch_set":1,"id":"1a26ad4f_76068187","line":213,"updated":"2015-11-02 01:42:21.000000000","message":"this is a really generic name for a specific issue ...\n\nwhy do you not just return \"SUCCESS\" here, but rather use it later to exit with that status?  it seems like globals getting tangled up with too much state","commit_id":"4f249a98da0c620f57f558039e5cb49d9c3dcd94"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"184b58b19112cea57188aac537a1223ab4c7ec77","unresolved":false,"context_lines":[{"line_number":193,"context_line":"# Propose patch using COMMIT_MSG"},{"line_number":194,"context_line":"function send_patch {"},{"line_number":195,"context_line":"    local branch\u003d${1:-master}"},{"line_number":196,"context_line":"    local OUTPUT"},{"line_number":197,"context_line":"    local RET"},{"line_number":198,"context_line":"    local SUCCESS\u003d0"},{"line_number":199,"context_line":""}],"source_content_type":"text/x-sh","patch_set":2,"id":"1a26ad4f_c24a0926","line":196,"updated":"2015-11-03 03:40:23.000000000","message":"fine if these are locals, but they really should be lower-case or it gets confusing","commit_id":"f28ea1d05c91650b320b7528217cb38fe0c71066"}],"jenkins/scripts/propose_translation_update.sh":[{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"184b58b19112cea57188aac537a1223ab4c7ec77","unresolved":false,"context_lines":[{"line_number":191,"context_line":"# Propose patch to gerrit if there are changes."},{"line_number":192,"context_line":"send_patch \"$BRANCH\""},{"line_number":193,"context_line":"SUCCESS\u003d$?"},{"line_number":194,"context_line":"exit $SUCCESS"}],"source_content_type":"text/x-sh","patch_set":2,"id":"1a26ad4f_426599b2","line":194,"updated":"2015-11-03 03:40:23.000000000","message":"so i don\u0027t think you need this, the result of send_patch will be the exit code?","commit_id":"f28ea1d05c91650b320b7528217cb38fe0c71066"}]}