-
Notifications
You must be signed in to change notification settings - Fork 41
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
🤖📯 Octokit webhooks changed #751
base: main
Are you sure you want to change the base?
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.
The docs format has changed, the script needs to be updated to try to restore the most functionality
With the changes to the docs, there are no longer any example payloads available. I think the automatic updater should be retired, and a new script that opens an issue when a new event is found be put in it's place instead. I believe this would be the best going forward until #470 is resolved |
According to github/docs#22726, the payloads will eventually get restored. They have an internal issue tracking this. |
I found new changes on GitHub’s Webhooks Event Types & Payloads and want to let you know about it 👋🤖
I can't tell if the changes are fixes, features or breaking changes. You'll have to figure that out on yourself and adapt the commit messages accordingly to trigger the right release, see our commit message conventions.
If there are any new events added, follow the steps outlined in order to create the required schemas.
If there were only description updates, or new properties added, simply modify the appropriate schemas and commit them to this PR. Don't forget to generate the types as well.