drop remaining six imports

Author: branchvincent

requirements-base.txt

@@ -1,3 +1,2 @@
 requests>=1.0
 retrying>=1.3.3
-six>=1.10.0

scrapinghub/client/collections.py

@@ -1,5 +1,4 @@
-from six import string_types
-from six.moves import collections_abc
+from collections.abc import Iterable
 
 from ..hubstorage.collectionsrt import Collection as _Collection
 
@@ -183,7 +182,7 @@ def delete(self, keys):
         The method returns ``None`` (original method returns an empty generator).
         """
         if (not isinstance(keys, str) and
-                not isinstance(keys, collections_abc.Iterable)):
+                not isinstance(keys, Iterable)):
             raise ValueError("You should provide string key or iterable "
                              "object providing string keys")
         self._origin.delete(keys)

scrapinghub/client/frontiers.py

@@ -1,8 +1,6 @@
 from functools import partial
 from collections import defaultdict
 
-from six import string_types
-
 from ..hubstorage.frontier import Frontier as _Frontier
 from ..hubstorage.utils import urlpathjoin
 

scrapinghub/client/proxy.py

@@ -1,4 +1,3 @@
-import six
 import json
 
 from ..hubstorage import ValueTooLarge as _ValueTooLarge
@@ -165,7 +164,7 @@ def iter(self):
         :return: an iterator over key/value pairs.
         :rtype: :class:`collections.abc.Iterable`
         """
-        return next(self._origin.apiget()).items()
+        return iter(next(self._origin.apiget()).items())
 
 
 def _format_iter_filters(params):

scrapinghub/client/utils.py

@@ -5,8 +5,6 @@
 import warnings
 from codecs import decode
 
-import six
-
 
 class LogLevel:
     DEBUG = logging.DEBUG

scrapinghub/hubstorage/batchuploader.py

@@ -3,7 +3,6 @@
 import random
 import logging
 import warnings
-import six
 from queue import Queue
 from io import BytesIO
 from gzip import GzipFile

scrapinghub/hubstorage/resourcetype.py

@@ -3,9 +3,8 @@
 import socket
 import time
 
-import six
 import requests.exceptions as rexc
-from six.moves import collections_abc
+from collections.abc import MutableMapping
 
 from .utils import urlpathjoin, xauth
 from .serialization import jlencode, jldecode, mpdecode
@@ -226,7 +225,7 @@ def stats(self):
         return next(self.apiget('stats', chunk_size=STATS_CHUNK_SIZE))
 
 
-class MappingResourceType(ResourceType, collections_abc.MutableMapping):
+class MappingResourceType(ResourceType, MutableMapping):
 
     _cached = None
     ignore_fields = ()

scrapinghub/hubstorage/serialization.py

@@ -1,4 +1,3 @@
-import six
 from json import dumps, loads
 from datetime import datetime
 

scrapinghub/hubstorage/utils.py

@@ -1,4 +1,3 @@
-import six
 import time
 from queue import Empty
 
@@ -80,7 +79,7 @@ class iterqueue:
 
     it exposes an attribute "count" with the number of messages read
 
-    >>> from six.moves.queue import Queue
+    >>> from queue import Queue
     >>> q = Queue()
     >>> for x in range(10):
     ...     q.put(x)

setup.py

@@ -26,7 +26,7 @@
     platforms=['Any'],
     packages=['scrapinghub', 'scrapinghub.client', 'scrapinghub.hubstorage'],
     package_data={'scrapinghub': ['VERSION']},
-    install_requires=['requests>=1.0', 'retrying>=1.3.3', 'six>=1.10.0'],
+    install_requires=['requests>=1.0', 'retrying>=1.3.3'],
     extras_require={'msgpack': mpack_required},
     python_requires='>=3.8',
     classifiers=[