pagure

#176 Flag pull-requests
Merged by . Opened by pingou.
flag  into  master

Download 176.patch
no initial comment

This comment seems incorrect. We are creating a repo?

Do you want query.one() here instead?

One or two small comments above, but otherwise :+1:

Wrong copy/pasta, will adjust :) (likely in more places)

.one() raises an exception if it does not find something, while .first() return None in those situation, so I like .first() and then I normally check if it returned something, rather than putting the method in a try/except block

Thanks for the review @ralph
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.