Skip to content

Commit

Permalink
[Fix] pr inference
Browse files Browse the repository at this point in the history
  • Loading branch information
Green-Ranger11 authored and ljharb committed Jan 5, 2022
1 parent b39a171 commit 67b119f
Show file tree
Hide file tree
Showing 3 changed files with 10 additions and 32 deletions.
2 changes: 1 addition & 1 deletion bin/can-merge
Original file line number Diff line number Diff line change
Expand Up @@ -115,7 +115,7 @@ function outputStatus(response) {
if (NODE_ENV === 'DEBUG') {
console.log(JSON.stringify(response, null, 2));
}
const prs = parsePullRequest(response.repository);
const prs = parsePullRequest(response);
if (prs.length === 0) {
console.info(chalk.redBright(`⚠ This remote repository does not contain any pull requests matching sha: ${args.sha}`));
process.exitCode = 1;
Expand Down
36 changes: 8 additions & 28 deletions utils/buildQuery.js
Original file line number Diff line number Diff line change
Expand Up @@ -50,36 +50,16 @@ module.exports = function buildQuery({
}) {
return `
{
repository(owner: "${owner}", name: "${name}") {
branchProtectionRules(first: 10) {
nodes {
requiresStatusChecks
requiredApprovingReviewCount
requiredStatusCheckContexts
}
}
${pr ? `pullRequest(number: ${pr}) {
${pullRequestQuery()}
}` : sha ? `commit: object(expression: "${sha}") {
... on Commit {
associatedPullRequests(first:50){
edges{
node{
${pullRequestQuery()}
}
}
}
}
}` : `pullRequests(first: 100) {
nodes {
search(query: "is:open is:pr repo:${owner}/${name} ${sha ? `sha:${sha}` : ''} ${pr ? `pr: ${pr}` : ''}", type:ISSUE, first: 100) {
issueCount
edges {
node {
... on PullRequest {
${pullRequestQuery()}
}
}
}`}
}
rateLimit {
cost
remaining
}
}
}
`;
`;
};
4 changes: 1 addition & 3 deletions utils/parsePullRequest.js
Original file line number Diff line number Diff line change
@@ -1,9 +1,7 @@
'use strict';

module.exports = function parsePullRequest(response) {
return response.pullRequest
? [response.pullRequest]
: response.commit?.associatedPullRequests.edges.flatMap(({ node }) => node)
return response.search.edges.flatMap(({ node }) => node)
|| response.pullRequests?.nodes
|| [];
};

0 comments on commit 67b119f

Please sign in to comment.