Skip to content

Commit d95e8bf

Browse files
committed
Bug 2009019 - Ensure we don't reference 'event' context when rendering actions
This context is specific to Github events and is not present for actions.
1 parent f36105c commit d95e8bf

File tree

1 file changed

+5
-4
lines changed

1 file changed

+5
-4
lines changed

.taskcluster.yml

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -346,11 +346,13 @@ tasks:
346346
$merge:
347347
- trustDomain: enterprise
348348
isPullRequest: false
349-
eventType: '${tasks_for[7:]}' # strip out 'github-'
350-
eventAction: '${event["action"]}' # empty string if 'action' doesn't exist
349+
- $if: 'tasks_for[:6] == "github"'
350+
then:
351+
ownTaskId: {$eval: as_slugid("decision_task")}
352+
eventType: '${tasks_for[7:]}' # strip out 'github-'
353+
eventAction: '${event["action"]}' # empty string if 'action' doesn't exist
351354
- $switch:
352355
'tasks_for == "github-push"':
353-
ownTaskId: {$eval: as_slugid("decision_task")}
354356
ownerEmail: '${event.pusher.email}'
355357
baseRepoUrl: '${event.repository.html_url}'
356358
repoUrl: '${event.repository.html_url}'
@@ -359,7 +361,6 @@ tasks:
359361
baseRev: '${event.before}'
360362
headRev: '${event.after}'
361363
'tasks_for[:19] == "github-pull-request"':
362-
ownTaskId: {$eval: as_slugid("decision_task")}
363364
ownerEmail: '${event.pull_request.user.login}@users.noreply.github.com'
364365
baseRepoUrl: '${event.pull_request.base.repo.html_url}'
365366
repoUrl: '${event.pull_request.head.repo.html_url}'

0 commit comments

Comments
 (0)