-
Notifications
You must be signed in to change notification settings - Fork 128
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
Add support for pull_request
events
#9
Comments
The problem is that the action is executed on the I'm planning to fix this though :) |
pull_request
event
Ah, that makes sense. It should fail the step regardless though, no? Or maybe just explode if ran on an unsupported event altogether |
Yeah, I expected this to be the current behavior, but it looks like I removed the check at some point 😅 I'll add it back as a temporary solution and will think about how the |
pull_request
eventpull_request
event
pull_request
eventpull_request
events
Released in v0.6.0 |
Oh, that was quick! Thanks 🎉 |
Probably something I messed up...
Both eslint and prettier says they found an error, but the step didn't fail and no annotation was added.
From jestjs/jest#9367
The text was updated successfully, but these errors were encountered: