)]}'
{"jenkins/scripts/sync_openstack_ansible_common_files.sh":[{"author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"change_message_id":"b2aa9f5572c91a13206c0cddcf72966da5f998f8","unresolved":false,"context_lines":[{"line_number":43,"context_line":"for x in ${files_to_sync[@]}; do"},{"line_number":44,"context_line":"    # If the target repo does not have such a file already"},{"line_number":45,"context_line":"    # then it\u0027s probably best to leave it alone."},{"line_number":46,"context_line":"    [[ ! -e ${OSA_PROJECT}/${x} ]] \u0026\u0026 continue"},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"    # Skip the project if it is in the excluded list"},{"line_number":49,"context_line":"    check_and_ignore ${OSA_PROJECT} \u0026\u0026 continue"}],"source_content_type":"text/x-sh","patch_set":2,"id":"7f287b81_a8180775","line":46,"range":{"start_line":46,"start_character":4,"end_line":46,"end_character":32},"updated":"2017-08-22 12:05:08.000000000","message":"What about moving your check up here and duplicating lines 44-46 with a similar content?\n\nThat would make it a bit clearer to understand what\u0027s going on.","commit_id":"77e085848963aeb0bd6903e2ae6f78c3e71bcad0"},{"author":{"_account_id":23163,"name":"Markos Chandras (hwoarang)","email":"markos@chandras.me","username":"mchandras"},"change_message_id":"1091514678d967316e8af0004e2c5dfb15ecc515","unresolved":false,"context_lines":[{"line_number":43,"context_line":"for x in ${files_to_sync[@]}; do"},{"line_number":44,"context_line":"    # If the target repo does not have such a file already"},{"line_number":45,"context_line":"    # then it\u0027s probably best to leave it alone."},{"line_number":46,"context_line":"    [[ ! -e ${OSA_PROJECT}/${x} ]] \u0026\u0026 continue"},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"    # Skip the project if it is in the excluded list"},{"line_number":49,"context_line":"    check_and_ignore ${OSA_PROJECT} \u0026\u0026 continue"}],"source_content_type":"text/x-sh","patch_set":2,"id":"7f287b81_88754b9b","line":46,"range":{"start_line":46,"start_character":4,"end_line":46,"end_character":32},"in_reply_to":"7f287b81_48564309","updated":"2017-08-22 12:07:46.000000000","message":"oh I guess you meant something like\n\n[[ ! -e ${x} ]] \u0026\u0026 continue\n\nso scratch my last comment.","commit_id":"77e085848963aeb0bd6903e2ae6f78c3e71bcad0"},{"author":{"_account_id":23163,"name":"Markos Chandras (hwoarang)","email":"markos@chandras.me","username":"mchandras"},"change_message_id":"1a6cadcd56db3477dc286ae1d6eba0b299dae549","unresolved":false,"context_lines":[{"line_number":43,"context_line":"for x in ${files_to_sync[@]}; do"},{"line_number":44,"context_line":"    # If the target repo does not have such a file already"},{"line_number":45,"context_line":"    # then it\u0027s probably best to leave it alone."},{"line_number":46,"context_line":"    [[ ! -e ${OSA_PROJECT}/${x} ]] \u0026\u0026 continue"},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"    # Skip the project if it is in the excluded list"},{"line_number":49,"context_line":"    check_and_ignore ${OSA_PROJECT} \u0026\u0026 continue"}],"source_content_type":"text/x-sh","patch_set":2,"id":"7f287b81_48564309","line":46,"range":{"start_line":46,"start_character":4,"end_line":46,"end_character":32},"in_reply_to":"7f287b81_a8180775","updated":"2017-08-22 12:06:41.000000000","message":"Actually the \u0027cp\u0027 command has to be after line 49 in case this particular project needs to be skipped on that job.","commit_id":"77e085848963aeb0bd6903e2ae6f78c3e71bcad0"}]}