Mercurial > p > roundup > code

changeset 8192:b4d7f9358ba6

Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
chore(ruff): return/suppress boolen directly I suppressed one 'if bool: return true else return false' so I could see in codecov that code was returning both true and false.
author John Rouillard <rouilj@ieee.org>
date Wed, 11 Dec 2024 13:29:11 -0500
parents 30818cc18058
children d06a140eab4d
files roundup/cgi/client.py roundup/rest.py
diffstat 2 files changed, 3 insertions(+), 4 deletions(-) [+]
line wrap: on
line diff
--- a/roundup/cgi/client.py	Wed Dec 11 13:19:49 2024 -0500
+++ b/roundup/cgi/client.py	Wed Dec 11 13:29:11 2024 -0500
@@ -1396,9 +1396,8 @@
         # allowing credentials. See:
         # https://developer.mozilla.org/en-US/docs/Web/HTTP/CORS
         # under Credentials Requests and Wildcards
-        if (allowed_origins and allowed_origins[0] == '*'
-            and not credentials):
-            return True
+        return (allowed_origins and allowed_origins[0] == '*'
+                and not credentials)
 
         return False
 
--- a/roundup/rest.py	Wed Dec 11 13:19:49 2024 -0500
+++ b/roundup/rest.py	Wed Dec 11 13:29:11 2024 -0500
@@ -239,7 +239,7 @@
                 return False
             have_etag_match = True
 
-    if have_etag_match:
+    if have_etag_match:  # noqa: SIM103  leave for coverage reports
         return True
     else:
         return False
Roundup Issue Tracker: http://roundup-tracker.org/