-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Implement installation_target webhook event type #2829
Implement installation_target webhook event type #2829
Conversation
Codecov Report
@@ Coverage Diff @@
## master #2829 +/- ##
=======================================
Coverage 98.06% 98.06%
=======================================
Files 136 136
Lines 12279 12281 +2
=======================================
+ Hits 12041 12043 +2
Misses 162 162
Partials 76 76
|
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.
Thank you, @RickleAndMortimer !
Just a few tweaks, please, then we should be ready for a second LGTM+Approval from any other contributor to this repo before merging.
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.
Thank you, @RickleAndMortimer !
LGTM.
Awaiting second LGTM+Approval from any other contributor to this repo before merging.
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
Thank you, @raynigon ! |
When can we expect a release to be published with this change? |
@mnafees - it would be nice to get a couple outstanding PRs merged in before the release since we already have breaking changes. Would you be interested in reviewing these two PRs? |
Implements the installation_target webhook event type.
Description from the API: