Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix actor and throw if required level is not met #3

Closed
wants to merge 2 commits into from

Conversation

skjnldsv
Copy link
Contributor

@skjnldsv skjnldsv commented Jul 24, 2021

Hi @xrkffgg! Great action!

I added support for everything now.
It will just always check the github actor.
So stuff like release user, comment editor, pull_request opener, workflow_dispatch initiator... etc

Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
src/main.js Outdated Show resolved Hide resolved
skjnldsv referenced this pull request in skjnldsv/check-actor-permission Jul 24, 2021
Update check-permission.yml
Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
@skjnldsv skjnldsv changed the title Add release support and throw if required level is not met Fix actor and throw if required level is not met Jul 24, 2021
@skjnldsv skjnldsv closed this Jul 24, 2021
@skjnldsv
Copy link
Contributor Author

in favor of #4

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant