-
Notifications
You must be signed in to change notification settings - Fork 141
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 the 'PushEvent' webhook and associated PushCommit object #386
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
cc @robbecker-wf is it possible to get this into a release? |
Looking now... sorry for the delay. |
Released in version 9.18.0 https://github.com/SpinlockLabs/github.dart/releases/tag/9.18.0 |
Hi @robbecker-wf, I noticed there is a discrepancy in the |
Hmm good question. Would it be easier for Google if we just reverted? I don't see other issues filed with this problem, but that doesn't mean there aren't people out there affected. |
Can you revert the change but release the package under the same version as that with the change? That way no one has to update and technically have two versions with the same code? |
This would probably be best if possible. I would also rather revert if we can. |
Once a version is published to the pub server, it's there. So the revert will have to be 9.19.0 |
Reverted in #387 |
PushGitCommit
that contains some data for the list of commits and head commit forPushEvent