Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Commit 0de1544

Browse filesBrowse files
authored
Merge pull request #798 from ncino/feat/using-rest-for-push
feat: updating push event trigger to use rest API (OctoKit) vs push event
2 parents baa1b23 + c3ab7fd commit 0de1544
Copy full SHA for 0de1544

File tree

Expand file treeCollapse file tree

3 files changed

+260
-124
lines changed
Filter options
Expand file treeCollapse file tree

3 files changed

+260
-124
lines changed

‎src/action.mjs

Copy file name to clipboardExpand all lines: src/action.mjs
+14-6Lines changed: 14 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -22,13 +22,21 @@ const getCommitDepth = () => {
2222
return Number.isNaN(commitDepth) ? null : Math.max(commitDepth, 0)
2323
}
2424

25-
const getPushEventCommits = () => {
26-
const mappedCommits = eventContext.payload.commits.map((commit) => ({
27-
message: commit.message,
28-
hash: commit.id,
29-
}))
25+
const getPushEventCommits = async () => {
26+
const octokit = getOctokit(getInput('token'))
27+
const { owner, repo } = eventContext.issue
28+
const { before } = eventContext.payload
29+
const { data: commits } = await octokit.rest.repos.listCommits({
30+
owner,
31+
repo,
32+
sha: before,
33+
per_page: 100,
34+
})
3035

31-
return mappedCommits
36+
return commits.map((commit) => ({
37+
message: commit.commit.message,
38+
hash: commit.sha,
39+
}))
3240
}
3341

3442
const getPullRequestEventCommits = async () => {

0 commit comments

Comments
0 (0)
Morty Proxy This is a proxified and sanitized view of the page, visit original site.