six lint

avishalom · avishalom · commit 181d4c0e5a48 · 2018-03-18T23:37:39.000-04:00
diff --git a/firebase_admin/dynamic_links.py b/firebase_admin/dynamic_links.py
@@ -88,7 +88,7 @@ def __init__(self, event_stats):
         if not isinstance(event_stats, (list, tuple)):
             raise ValueError('Invalid data argument: {0}. Must be a list or tuple'
                              .format(event_stats))
-        if len(event_stats) > 0 and not isinstance(event_stats[0], EventStats):
+        if event_stats and not isinstance(event_stats[0], EventStats):
             raise ValueError('Invalid data argument: elements of event stats must be' +
                              ' "EventStats", found{}'.format(type(event_stats[0])))
         self._stats = event_stats
@@ -114,10 +114,12 @@ def __init__(self, app):
 
     def _populated_request(self, url, options):
         days = options.duration_days
-        #pylint: disable-msg=too-many-arguments
-        #this is due to six.urllib_parse incompatibility 
-        url_quoted = urllib_parse.quote(url, "")
-        #pylint: enable-msg=too-many-arguments
+        #pylint: disable=too-many-function-args
+        # This is due to six.urllib_parse incompatibility
+        #pylint: disable=redundant-keyword-arg
+        url_quoted = urllib_parse.quote(url, safe="")
+        #pylint: enable=redundant-keyword-arg
+        #pylint: enable=too-many-function-args
         return self._links_request.format(url_quoted, days)
 
     def get_stats(self, url, options):
