Skip to content

Commit c945be2

Browse files
authored
Merge pull request #11 from DEFRA/489130
Only look for workflow_run name in workflow_run events
2 parents 89deaf7 + a4f0f3f commit c945be2

File tree

1 file changed

+10
-12
lines changed

1 file changed

+10
-12
lines changed

src/listeners/github/message-handler.js

Lines changed: 10 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -15,18 +15,16 @@ const githubWebhooks = new Set([
1515
])
1616

1717
const shouldProcessCreateWorkflows = (message) => {
18-
const repoName = message.repository?.name
19-
const workflowName = message.workflow_run.name
20-
21-
const infraDevTfSvcInfraRun =
22-
repoName === gitHubRepoConfig.cdpTfSvcInfra &&
23-
workflowName.toLocaleLowerCase().includes('infra-dev')
24-
25-
return (
26-
message.github_event === 'workflow_run' &&
27-
githubWebhooks.has(repoName) &&
28-
!infraDevTfSvcInfraRun
29-
)
18+
if (message?.github_event === 'workflow_run') {
19+
const repoName = message.repository?.name
20+
const workflowName = message.workflow_run?.name
21+
const infraDevTfSvcInfraRun =
22+
repoName === gitHubRepoConfig.cdpTfSvcInfra &&
23+
workflowName.toLocaleLowerCase().includes('infra-dev')
24+
25+
return githubWebhooks.has(repoName) && !infraDevTfSvcInfraRun
26+
}
27+
return false
3028
}
3129

3230
const shouldProcessPortalJourneyTests = (message) => {

0 commit comments

Comments
 (0)