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

Add a Changes object to the PullRequestEvent object so we can see what changed in edited PR events #390

Merged

Conversation

ricardoamador
Copy link
Collaborator

Adding a Changes object to the PullRequestEvent object so that we can determine what changed.

See this doc: https://docs.github.com/en/webhooks/webhook-events-and-payloads?actionType=edited#pull_request

CaseyHillers
CaseyHillers previously approved these changes Nov 8, 2023
lib/src/common/model/changes.dart Outdated Show resolved Hide resolved
lib/src/server/hooks.dart Outdated Show resolved Hide resolved
@ricardoamador
Copy link
Collaborator Author

@robbecker-wf @robrbecker friendly ping.

@robrbecker robrbecker added unreleased semver:minor Non-Breaking Public API Changes (additions) labels Nov 14, 2023
@robrbecker robrbecker merged commit 01a87bf into SpinlockLabs:master Nov 14, 2023
5 checks passed
@robbecker-wf
Copy link
Contributor

@ricardoamador
Copy link
Collaborator Author

Thank you @robbecker-wf!

@ricardoamador
Copy link
Collaborator Author

@robbecker-wf will you be pushing this to pubdev as well?

@robrbecker
Copy link
Member

robrbecker commented Nov 15, 2023 via email

@ricardoamador
Copy link
Collaborator Author

ricardoamador commented Nov 15, 2023

I don't see it here: https://pub.dev/packages/github
Am I looking in the wrong place?

@robrbecker
Copy link
Member

Oh, the publish action failed! https://github.com/SpinlockLabs/github.dart/actions/runs/6867506661
I'll have to look into that, but for now I just manually published.

@ricardoamador
Copy link
Collaborator Author

Okay thank you Rob! I was just making sure I was not looking in the wrong place. Appreciate it!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
released semver:minor Non-Breaking Public API Changes (additions)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants