diff options
author | Teddy Wing | 2018-04-18 22:02:27 +0200 |
---|---|---|
committer | Teddy Wing | 2018-04-18 22:02:27 +0200 |
commit | a2cacedf44d2488528cae45ecc1b48483903be62 (patch) | |
tree | 1e9a4ca8e96e085d0c53f4a170c66f02d1da57c7 | |
parent | c3f749355ecde0f1d79524fa9b00f47e32e1437f (diff) | |
download | redprine-a2cacedf44d2488528cae45ecc1b48483903be62.tar.bz2 |
update_redmine_statuses(): Don't update if there are no issues
Previously this would run `update_redmine_status()` even if
`$issue_ids_and_urls` was empty, which would make an unnecessary request
without the required data.
-rwxr-xr-x | redprine | 8 |
1 files changed, 5 insertions, 3 deletions
@@ -140,9 +140,11 @@ function issue_numbers_and_pull_request_urls () { function update_redmine_statuses () { local issue_ids_and_urls="$1" - for id_and_url in "$issue_ids_and_urls"; do - update_redmine_status "$id_and_url" - done + if [ ! -z "$issue_ids_and_urls" ]; then + for id_and_url in "$issue_ids_and_urls"; do + update_redmine_status "$id_and_url" + done + fi } function update_redmine_status () { |