STACKITGit/cli
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Ensure PR does not panic when stateReason is requested

Browse source
This commit is contained in:
William Martin 2024-07-11 16:30:35 +02:00
parent 0df5596512
commit 172a9de2fe
1 changed files with 12 additions and 8 deletions
Download patch file Download diff file Expand all files Collapse all files

20
api/query_builder.go
View file

@ -249,7 +249,7 @@ func RequiredStatusCheckRollupGraphQL(prID, after string, includeEvent bool) str
}`), afterClause, prID, eventField)
}
var IssueFields = []string{
var sharedIssuePRFields = []string{
"assignees",
"author",
"body",
@ -268,10 +268,20 @@ var IssueFields = []string{
"title",
"updatedAt",
"url",
}
// Some fields are only valid in the context of issues.
// They need to be enumerated separately in order to be filtered
// from existing code that expects to be able to pass Issue fields
// to PR queries, e.g. the PullRequestGraphql function.
var issueOnlyFields = []string{
"isPinned",
"stateReason",
}
var PullRequestFields = append(IssueFields,
var IssueFields = append(sharedIssuePRFields, issueOnlyFields...)
var PullRequestFields = append(sharedIssuePRFields,
"additions",
"autoMergeRequest",
"baseRefName",
@ -299,12 +309,6 @@ var PullRequestFields = append(IssueFields,
"statusCheckRollup",
)
// Some fields are only valid in the context of issues.
var issueOnlyFields = []string{
"isPinned",
"stateReason",
}
// IssueGraphQL constructs a GraphQL query fragment for a set of issue fields.
func IssueGraphQL(fields []string) string {
var q []string