Skip to content

Commit 7016d44

Browse files
committed
fix error handling
1 parent 549fcbd commit 7016d44

File tree

1 file changed

+10
-9
lines changed

1 file changed

+10
-9
lines changed

tools/amplify-preview/main.go

+10-9
Original file line numberDiff line numberDiff line change
@@ -67,12 +67,12 @@ func main() {
6767
// Check if Amplify branch is already connected to one of the Amplify Apps
6868
branch, err := amp.FindExistingBranch(ctx, *gitBranchName)
6969
if err != nil {
70-
logger.Error("failed to lookup branch", logKeyBranchName, *gitBranchName, "error", err)
71-
os.Exit(1)
72-
}
70+
if !(errors.Is(err, errNoJobForBranch) && *createBranches) {
71+
logger.Error("failed to lookup branch", logKeyBranchName, *gitBranchName, "error", err)
72+
os.Exit(1)
73+
}
7374

74-
// If branch wasn't found, and branch creation enabled - create new branch
75-
if branch == nil && *createBranches {
75+
// If branch wasn't found, and branch creation enabled - create new branch
7676
branch, err = amp.CreateBranch(ctx, *gitBranchName)
7777
if err != nil {
7878
logger.Error("failed to create branch", logKeyBranchName, *gitBranchName, "error", err)
@@ -83,11 +83,12 @@ func main() {
8383
// check if existing branch was/being already deployed
8484
job, err := amp.GetJob(ctx, branch, nil)
8585
if err != nil {
86-
logger.Error("failed to get amplify job", logKeyBranchName, *gitBranchName, "error", err)
87-
os.Exit(1)
88-
}
86+
if !(errors.Is(err, errNoJobForBranch) && *createBranches) {
87+
logger.Error("failed to get amplify job", logKeyBranchName, *gitBranchName, "error", err)
88+
os.Exit(1)
89+
}
8990

90-
if errors.Is(err, errNoJobForBranch) && *createBranches {
91+
// if job not found and branch was just created - start new job
9192
job, err = amp.StartJob(ctx, branch)
9293
if err != nil {
9394
logger.Error("failed to start amplify job", logKeyBranchName, *gitBranchName, "error", err)

0 commit comments

Comments
 (0)