pagure-dist-git

#83 dist_git_auth: fix pr_only check logic
Merged by pingou. Opened by jlanda.
jlanda/pagure-dist-git fix-pr-only  into  master

Download 83.patch

This fixes #84

rebased onto 55cb5156520ae6b66782c19564463e77989f539c

2 new commits added

  • tests: test project level pull requests only
  • dist_git_auth: fix pr_only check logic

I'll need to run the tests locally before merging this but from a quick looks good.

I wonder if we could work something in the indentation to make the logical block clearer (indent within the parenthesis?), I'll likely play around with this when testing it :)

Ok tests are passing locally and the change looks fine, so let's get this in.
I'll open a new PR running black on the code :)

Pull-Request has been merged by pingou
Metadata
None
No Tags
Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.
Red Hat Logo

Fedora is sponsored by Red Hat. Learn more about the relationship between Red Hat and Fedora »

© Red Hat, Inc. and others.