Skip to content
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

Open
wants to merge 200 commits into
base: main
Choose a base branch
from
Open

🤖📯 Octokit webhooks changed #751

wants to merge 200 commits into from

Conversation

octokitbot
Copy link
Collaborator

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.

@octokitbot octokitbot added the Type: Maintenance Any dependency, housekeeping, and clean up Issue or PR label Dec 15, 2022
@ghost ghost added this to Inbox in JS Dec 15, 2022
@ghost ghost moved this from Inbox to Maintenance in JS Dec 15, 2022
Copy link
Member

@wolfy1339 wolfy1339 left a 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

@wolfy1339
Copy link
Member

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

@wolfy1339
Copy link
Member

According to github/docs#22726, the payloads will eventually get restored. They have an internal issue tracking this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Maintenance Any dependency, housekeeping, and clean up Issue or PR
Projects
Status: 🏗 In progress
JS
  
Maintenance
Development

Successfully merging this pull request may close these issues.

None yet

3 participants