changeset 7834:dbeb6bf056c7

chore(lint): _ unused var; replace assert wth if/raise; whitespace
author John Rouillard <rouilj@ieee.org>
date Sun, 24 Mar 2024 15:29:55 -0400
parents b68a1d8fd5d9
children c6fcc8ba478a
files roundup/cgi/wsgi_handler.py
diffstat 1 files changed, 4 insertions(+), 2 deletions(-) [+]
line wrap: on
line diff
--- a/roundup/cgi/wsgi_handler.py	Sun Mar 24 15:25:53 2024 -0400
+++ b/roundup/cgi/wsgi_handler.py	Sun Mar 24 15:29:55 2024 -0400
@@ -27,6 +27,7 @@
         'a given amount of time ("rate limiting")',
     )
 
+
 class Headers(object):
     """ Idea more or less stolen from the 'apache.py' in same directory.
         Except that wsgi stores http headers in environment.
@@ -68,7 +69,7 @@
 
     def start_response(self, headers, response_code):
         """Set HTTP response code"""
-        message, explain = BaseHTTPRequestHandler.responses[response_code]
+        message, _explain = BaseHTTPRequestHandler.responses[response_code]
         self.__wfile = self.__start_response('%d %s' % (response_code,
                                                         message), headers)
 
@@ -81,7 +82,8 @@
 class RequestDispatcher(object):
     def __init__(self, home, debug=False, timing=False, lang=None,
                  feature_flags=None):
-        assert os.path.isdir(home), '%r is not a directory' % (home,)
+        if not os.path.isdir(home):
+            raise ValueError('%r is not a directory' % (home,))
         self.home = home
         self.debug = debug
         self.timing = timing

Roundup Issue Tracker: http://roundup-tracker.org/