drop unnecessary backwards compatibility (remove six)

beeline/patch/test_urllib.py

@@ -1,13 +1,11 @@
 from __future__ import absolute_import
-import six
 import unittest
 from mock import Mock, patch
 
 import urllib
 
 
 class TestUrllibPatch(unittest.TestCase):
-    @unittest.skipIf(six.PY2, "urllib not compatible with python2")
     def test_request_fn_injects_headers_and_returns(self):
         from beeline.patch.urllib import _urllibopen  # pylint: disable=bad-option-value,import-outside-toplevel
 

beeline/propagation/__init__.py

@@ -1,4 +1,3 @@
-import six
 from abc import ABCMeta, abstractmethod
 import beeline
 
@@ -23,8 +22,7 @@ def __init__(self, trace_id, parent_id, trace_fields={}, dataset=None):
         self.dataset = dataset
 
 
-@six.add_metaclass(ABCMeta)
-class Request(object):
+class Request(object, metaclass=ABCMeta):
     '''
     beeline.propagation.Request is an abstract class that defines the interface that should
     be used by middleware to pass request information into http_trace_parser_hooks. It should

beeline/propagation/honeycomb.py

@@ -2,7 +2,7 @@
 from beeline.propagation import PropagationContext
 import base64
 import json
-from six.moves.urllib.parse import quote, unquote
+from urllib.parse import quote, unquote
 
 
 def http_trace_parser_hook(request):

pylint.rc

@@ -537,7 +537,7 @@ init-import=no
 
 # List of qualified module names which can have objects that can redefine
 # builtins.
-redefining-builtins-modules=six.moves,past.builtins,future.builtins,io,builtins
+redefining-builtins-modules=past.builtins,future.builtins,io,builtins
 
 
 [FORMAT]