paulononato/backstage
1
0
Fork 0
Code Issues Pull Requests Actions 22 Packages Projects Releases Wiki Activity

Merge pull request #9053 from backstage/benjdlambert-patch-1

Browse Source 
goalie: cron is failing due to no user in some events
This commit is contained in:
Patrik Oldsberg
2022-01-20 14:18:40 +01:00
committed by GitHub
parent a370706a18 21885379c1
commit bed2da46af
1 changed files with 4 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files
scripts/goalie-labels.js
+4 -4
View File
@@ -50,9 +50,9 @@ const getRepoEvents = async ({ github, context, pull_number }) => {
updated_at: submitted_at,
type: 'review',
})),
...commits.map(({ commit, author }) => ({
user: author,
updated_at: commit.author.date,
...commits.map(({ commit, author, committer }) => ({
user: author || committer,
updated_at: commit.author.date || commit.committer.date,
type: 'commit',
})),
...pullComments.map(({ user, updated_at }) => ({
@@ -69,7 +69,7 @@ const getRepoEvents = async ({ github, context, pull_number }) => {
return events
.sort((a, b) => new Date(a.updated_at) - new Date(b.updated_at))
.filter(({ user }) => !user.login.includes('[bot]'));
.filter(({ user }) => (user ? !user.login.includes('[bot]') : false));
};
module.exports = async ({ github, context, core }) => {