limit-pull-requests checks PR count by who triggers the workflow rather than PR author

[karaarda]

github.context.actor is the initiator of the workflow. To get the original author of the PR github.context.payload.user.login should be user. Please refer to https://github.com/orgs/community/discussions/25502.

@ZhongRuoyu

[ZhongRuoyu]

Thanks for reporting; technically you're right, but is there a case where there is actual difference between the two, given that its intended use case is when github.event_name == 'pull_request_target' && github.event.action == 'opened'?

[karaarda]

Well, its intended use case is not documented and as this is a public repository anyone can include this action in their workflows like I did. It didn't play well in our approach where we would like to run our checks on 'opened', 'reopened' and 'synchronize'

Also if it is only ran on 'opened', doesn't it mean that one can just reopen the repo to walk around the PR limit?

I can open a PR with the fix if you agree it is a potential bug

[MikeMcQuaid]

I can open a PR with the fix if you agree it is a potential bug

Reviewing a PR will be easier to discuss, yes, please 👍🏻

[github-actions]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.