-
Notifications
You must be signed in to change notification settings - Fork 165
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/manual and scheduled runs #74
Fix/manual and scheduled runs #74
Conversation
Hello @rhysd |
Would <3 if this gets merged too as I'm also having issue #73 |
Thanks for this change! I made some edits that I think would be useful in this PR on your fork: ktrz#1. @rhysd we're using my changes successfully on our |
Remove references to payload as much as possible
@rhysd Just trying to catch your attention now that I've seen you working on this repo! This change is tested to work for If you think it looks good could we merge it in? Please let us know if you have any questions! |
hi @NathanielRN |
resolve #24
resolve #73
If there are not
head_commit
norpull_request
data in the action payload I've added a fallback to fetch the data about the current commit via GitHub REST API