fedora-qa / oraculum

Clone
Source Code
GIT

#67 Return empty list on ENOPROJECT from Pagure
Merged 3 years ago by frantisekz. Opened 3 years ago by frantisekz.
return_list_instead_of_none  into  master

file modified
+1 -1 
@@ -454,7 +454,7 @@ 
 

          return None
 

      # Filter out packages that, for some reason, do not exist in dist-git
 

      if "error_code" in resp_package_prs and resp_package_prs["error_code"] == "ENOPROJECT":
 

-         return None
 

+         return []
 

      if not "requests" in resp_package_prs:
 

          app.logger.error("Skipping PRs from package %s because Pagure returned invalid data" % package)
 

          return []  # Return early if pagure sent invalid data

Otherwise, we will keep asking for results again and again... and never get them.

:thumbsup:

Pull-Request has been merged by frantisekz
3 years ago
Metadata
None
No Tags
Changes Summary 1
+1 -1
file changed
oraculum/utils/dashboard.py
Powered by Pagure 5.13.3
Documentation • File an Issue • About • SSH Hostkey/Fingerprint
© Red Hat, Inc. and others.