Skip to content

Commit a13fefd

Browse files
authored
fix for long status names (#1907)
1 parent b38a438 commit a13fefd

File tree

1 file changed

+16
-10
lines changed

1 file changed

+16
-10
lines changed

libs/ci/github/github.go

+16-10
Original file line numberDiff line numberDiff line change
@@ -251,12 +251,18 @@ func (svc GithubService) SetStatus(prNumber int, status string, statusContext st
251251
if err != nil {
252252
log.Printf("error getting pull request : %v", err)
253253
return fmt.Errorf("error getting pull request : %v", err)
254-
255254
}
255+
256+
// previously was setting description as "statusContext" but
257+
// faced some issues with too long strings of > 140 chars:
258+
// 422 Validation Failed [{Resource:Status Field:description Code:custom Message:description is too long (maximum is 140 characters)}]
259+
// since description isn't shown in ui setting to blank for now
260+
description := ""
261+
256262
_, _, err = svc.Client.Repositories.CreateStatus(context.Background(), svc.Owner, svc.RepoName, *pr.Head.SHA, &github.RepoStatus{
257263
State: &status,
258264
Context: &statusContext,
259-
Description: &statusContext,
265+
Description: &description,
260266
})
261267
return err
262268
}
@@ -490,9 +496,9 @@ func ConvertGithubPullRequestEventToJobs(payload *github.PullRequestEvent, impac
490496
RequestedBy: *payload.Sender.Login,
491497
CommandEnvProvider: CommandEnvProvider,
492498
CommandRoleArn: cmdRole,
493-
StateRoleArn: stateRole,
499+
StateRoleArn: stateRole,
494500
StateEnvProvider: StateEnvProvider,
495-
CognitoOidcConfig: project.AwsCognitoOidcConfig,
501+
CognitoOidcConfig: project.AwsCognitoOidcConfig,
496502
SkipMergeCheck: skipMerge,
497503
})
498504
} else if *payload.Action == "opened" || *payload.Action == "reopened" || *payload.Action == "synchronize" {
@@ -516,9 +522,9 @@ func ConvertGithubPullRequestEventToJobs(payload *github.PullRequestEvent, impac
516522
RequestedBy: *payload.Sender.Login,
517523
CommandEnvProvider: CommandEnvProvider,
518524
CommandRoleArn: cmdRole,
519-
StateRoleArn: stateRole,
525+
StateRoleArn: stateRole,
520526
StateEnvProvider: StateEnvProvider,
521-
CognitoOidcConfig: project.AwsCognitoOidcConfig,
527+
CognitoOidcConfig: project.AwsCognitoOidcConfig,
522528
SkipMergeCheck: skipMerge,
523529
})
524530
} else if *payload.Action == "closed" {
@@ -542,9 +548,9 @@ func ConvertGithubPullRequestEventToJobs(payload *github.PullRequestEvent, impac
542548
RequestedBy: *payload.Sender.Login,
543549
CommandEnvProvider: CommandEnvProvider,
544550
CommandRoleArn: cmdRole,
545-
StateRoleArn: stateRole,
551+
StateRoleArn: stateRole,
546552
StateEnvProvider: StateEnvProvider,
547-
CognitoOidcConfig: project.AwsCognitoOidcConfig,
553+
CognitoOidcConfig: project.AwsCognitoOidcConfig,
548554
SkipMergeCheck: skipMerge,
549555
})
550556
} else if *payload.Action == "converted_to_draft" {
@@ -575,9 +581,9 @@ func ConvertGithubPullRequestEventToJobs(payload *github.PullRequestEvent, impac
575581
RequestedBy: *payload.Sender.Login,
576582
CommandEnvProvider: CommandEnvProvider,
577583
CommandRoleArn: cmdRole,
578-
StateRoleArn: stateRole,
584+
StateRoleArn: stateRole,
579585
StateEnvProvider: StateEnvProvider,
580-
CognitoOidcConfig: project.AwsCognitoOidcConfig,
586+
CognitoOidcConfig: project.AwsCognitoOidcConfig,
581587
SkipMergeCheck: skipMerge,
582588
})
583589
}

0 commit comments

Comments
 (0)