chore: fix flake8 warnings around isinstance use

Also add max-line-length to setup.cfg to avoid E501 flagging existing code
(and 100 is the standard default in most Python modules due to black/ruff)

Signed-off-by: Dominic Evans <[email protected]>

Author: dnwe

detect_secrets/plugins/artifactory.py

@@ -21,7 +21,7 @@ class ArtifactoryDetector(RegexBasedDetector):
 
     def verify(self, token, *args, **kwargs):
         try:
-            if type(token) == bytes:
+            if isinstance(token, bytes):
                 token = token.decode('UTF-8')
             headers = {'X-JFrog-Art-API': token}
             response = requests.get(

detect_secrets/plugins/github_enterprise.py

@@ -77,7 +77,7 @@ def __init__(self, ghe_instance=DEFAULT_GHE_INSTANCE, *args, **kwargs):
 
     def verify(self, token, *args, **kwargs):
         try:
-            if type(token) == bytes:
+            if isinstance(token, bytes):
                 token = token.decode('UTF-8')
             headers = {'Authorization': 'token %s' % token}
             response = requests.get(f'https://{self.ghe_instance}/api/v3', headers=headers)

detect_secrets/plugins/ibm_cloud_iam.py

@@ -42,7 +42,7 @@ def verify(self, token, *args, **kwargs):
 
 
 def verify_cloud_iam_api_key(apikey):  # pragma: no cover
-    if type(apikey) == bytes:
+    if isinstance(apikey, bytes):
         apikey = apikey.decode('UTF-8')
     headers = {
         'Content-Type': 'application/x-www-form-urlencoded',

setup.cfg

@@ -4,5 +4,8 @@ description-file = README.md
 [wheel]
 universal = True
 
+[flake8]
+max-line-length = 100
+
 [tool:pytest]
 norecursedirs=tests/testing