-
Notifications
You must be signed in to change notification settings - Fork 56
Parse Pagure PR events #2600
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
Parse Pagure PR events #2600
Conversation
|
Build succeeded. ✔️ pre-commit SUCCESS in 2m 11s |
nforro
left a comment
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
for more information, see https://pre-commit.ci
|
Build succeeded. ✔️ pre-commit SUCCESS in 2m 02s |
| logger.debug(f"Base project: {fork} owned by {self.base_repo_owner}") | ||
| return fork | ||
|
|
||
| def get_packages_config(self) -> Optional[PackageConfig]: |
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.
👀 Do we have both project × get_project and packages_config × get_packages_config? /o\
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.
yes I fear :D
|
Build succeeded (gate pipeline). ✔️ pre-commit SUCCESS in 2m 08s |
a72fc6b
into
packit:main
Fixes #2579
Related to packit/packit-service-fedmsg#115
Related to packit/deployment#608
RELEASE NOTES BEGIN
N/A
RELEASE NOTES END