Skip to content

Commit b09c3eb

Browse files
Copilotpelikhan
andauthored
perf: avoid extra PR fetch for merge-only update-pull-request
Agent-Logs-Url: https://github.com/github/gh-aw/sessions/340b546b-743d-45b6-ac7e-8ec5735124d3 Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
1 parent 0405d9e commit b09c3eb

2 files changed

Lines changed: 5 additions & 6 deletions

File tree

actions/setup/js/update_pull_request.cjs

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -89,12 +89,10 @@ async function executePRUpdate(github, context, prNumber, updateData) {
8989
}
9090

9191
if (Object.keys(apiData).length === 0) {
92-
const { data: pr } = await github.rest.pulls.get({
93-
owner: context.repo.owner,
94-
repo: context.repo.repo,
95-
pull_number: prNumber,
96-
});
97-
return pr;
92+
return {
93+
number: prNumber,
94+
html_url: `${context.serverUrl}/${context.repo.owner}/${context.repo.repo}/pull/${prNumber}`,
95+
};
9896
}
9997

10098
const { data: pr } = await github.rest.pulls.update({

actions/setup/js/update_pull_request.test.cjs

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -802,6 +802,7 @@ describe("update_pull_request.cjs - merge_base behavior", () => {
802802
repo: "testrepo",
803803
pull_number: 100,
804804
});
805+
expect(mockGithub.rest.pulls.get).not.toHaveBeenCalled();
805806
expect(mockGithub.rest.pulls.update).not.toHaveBeenCalled();
806807
});
807808

0 commit comments

Comments
 (0)