Skip to content

Commit

Permalink
Use pull_request.user, not actor to determine PR user (#15504)
Browse files Browse the repository at this point in the history
In most cases these are the same -- the one exception is when
(re)opening an issue, in which case the actor is going to be someone
with commit rights to a repo, and we don't want the mere act of
re-opening to cause a PR to run on self-hosted infrastructure as that
would be surprising (and potentially unsafe)
  • Loading branch information
ashb authored Apr 24, 2021
1 parent be421a6 commit be8d2b1
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -122,7 +122,7 @@ jobs:
"xinbinhuang",
"yuqian",
"eladkal"
]'), github.actor)
]'), github.event.pull_request.user.login)
) && github.repository == 'apache/airflow'
) && 'self-hosted' || 'ubuntu-20.04' }}
env:
Expand Down

0 comments on commit be8d2b1

Please sign in to comment.