)]}'
{"openstackclient/tests/functional/post_test_hook.sh":[{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"f0cb68994c11662f0ee053ed23bef73c3c97d794","unresolved":false,"context_lines":[{"line_number":16,"context_line":"        sudo chown jenkins:jenkins $BASE/logs/testrepository.subunit.gz $BASE/logs/testr_results.html.gz"},{"line_number":17,"context_line":"        sudo chmod a+r $BASE/logs/testrepository.subunit.gz $BASE/logs/testr_results.html.gz"},{"line_number":18,"context_line":"    elif [ -f .stestr/0 ]; then"},{"line_number":19,"context_line":"        sudo .tox/functional/bin/stestr last --subunit \u003e $WORKSPACE/testrepository.subunit"},{"line_number":20,"context_line":"        sudo mv $WORKSPACE/testrepository.subunit $BASE/logs/testrepository.subunit"},{"line_number":21,"context_line":"        sudo .tox/functional/bin/subunit2html $BASE/logs/testrepository.subunit $BASE/logs/testr_results.html"},{"line_number":22,"context_line":"        sudo gzip -9 $BASE/logs/testrepository.subunit"}],"source_content_type":"text/x-sh","patch_set":1,"id":"5f2577a7_e05e9674","line":19,"range":{"start_line":19,"start_character":8,"end_line":19,"end_character":90},"updated":"2017-09-15 03:39:05.000000000","message":"all but this is the same, why not have the common lines outside of the if?","commit_id":"3048fffd0c0802e9a94f56a9fa411f4fda870582"},{"author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"change_message_id":"8780fe59772c2b3358e13a205c09d5743ce124f2","unresolved":false,"context_lines":[{"line_number":16,"context_line":"        sudo chown jenkins:jenkins $BASE/logs/testrepository.subunit.gz $BASE/logs/testr_results.html.gz"},{"line_number":17,"context_line":"        sudo chmod a+r $BASE/logs/testrepository.subunit.gz $BASE/logs/testr_results.html.gz"},{"line_number":18,"context_line":"    elif [ -f .stestr/0 ]; then"},{"line_number":19,"context_line":"        sudo .tox/functional/bin/stestr last --subunit \u003e $WORKSPACE/testrepository.subunit"},{"line_number":20,"context_line":"        sudo mv $WORKSPACE/testrepository.subunit $BASE/logs/testrepository.subunit"},{"line_number":21,"context_line":"        sudo .tox/functional/bin/subunit2html $BASE/logs/testrepository.subunit $BASE/logs/testr_results.html"},{"line_number":22,"context_line":"        sudo gzip -9 $BASE/logs/testrepository.subunit"}],"source_content_type":"text/x-sh","patch_set":1,"id":"5f2577a7_a44e586c","line":19,"in_reply_to":"5f2577a7_e05e9674","updated":"2017-09-15 17:37:16.000000000","message":"because it falls through if neither are present... that\u0027s an elif, not else on line 18","commit_id":"3048fffd0c0802e9a94f56a9fa411f4fda870582"}]}