-
Notifications
You must be signed in to change notification settings - Fork 9
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
Migrate to org.fedoraproject.prod.pagure.git.receive #100
base: main
Are you sure you want to change the base?
Conversation
Signed-off-by: Nikola Forró <[email protected]>
Build succeeded. ✔️ pre-commit SUCCESS in 1m 48s |
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.
If no changes need to be done here: https://github.com/packit/packit-service-fedmsg/pull/100/files#diff-b7fd01b92d80cc4358198977ebcf33e52913f1395fd029b6decf71ba106f9496R73
LGTM
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.
I think the method itself needs some adjustments as well, right?
Oh, right, I missed that. Let me fix it. |
Actually, it looks like there is no way to differentiate between a PR event and a direct push event now. We also don't have a list of changed files. |
Migrate to org.fedoraproject.prod.pagure.git.receive Related to packit/packit-service#2305 Supersedes #100 Reviewed-by: Nikola Forró
No description provided.