Skip to content

Commit

Permalink
more-careful-deploy-error-reporting
Browse files Browse the repository at this point in the history
  • Loading branch information
cpsievert committed Aug 10, 2023
1 parent 25644a0 commit 87842f7
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions R/deploy-apps.R
Original file line number Diff line number Diff line change
Expand Up @@ -114,7 +114,7 @@ deploy_apps <- function(
if (inherits(deployment_worked, 'try-error')) {
return(1)
} else {
return(as.numeric(!deployment_worked))
return(as.numeric(!isTRUE(deployment_worked)))
}
}

Expand All @@ -137,7 +137,7 @@ deploy_apps <- function(
}
)

if (all(deploy_res == 0)) {
if ((!any(deploy_res == 1)) && length(deploy_res) == length(app_dirs)) {
# success!
message("No errors found when deploying apps")
return(invisible(NULL))
Expand Down

0 comments on commit 87842f7

Please sign in to comment.