style: 🚨 Address linter complaints

Author: jdrew82

diffsync/store/redis.py

@@ -2,8 +2,8 @@
 
 import copy
 import uuid
-from pickle import loads, dumps  # nosec
-from typing import List, Type, Union, TYPE_CHECKING, Set, Any, Optional, Dict
+from pickle import dumps, loads  # nosec
+from typing import TYPE_CHECKING, Any, Dict, List, Optional, Set, Type, Union
 
 try:
     from redis import Redis
@@ -12,7 +12,7 @@
     print("Redis is not installed. Have you installed diffsync with redis extra? `pip install diffsync[redis]`")
     raise ierr
 
-from diffsync.exceptions import ObjectNotFound, ObjectStoreException, ObjectAlreadyExists
+from diffsync.exceptions import ObjectAlreadyExists, ObjectNotFound, ObjectStoreException
 from diffsync.store import BaseStore
 
 if TYPE_CHECKING:
@@ -168,7 +168,7 @@ def add(self, *, obj: "DiffSyncModel") -> None:
 
         existing_obj_binary = self._store.get(object_key)
         if existing_obj_binary:
-            existing_obj = loads(existing_obj_binary)
+            existing_obj = loads(existing_obj_binary)  # noqa: B301
             existing_obj_dict = existing_obj.dict()
 
             if existing_obj_dict != obj.dict():

examples/05-nautobot-peeringdb/adapter_peeringdb.py

@@ -2,14 +2,15 @@
 
 # pylint: disable=import-error,no-name-in-module
 import os
-import requests
-from slugify import slugify
+
 import pycountry
+import requests
 from models import RegionModel, SiteModel
+from slugify import slugify
+
 from diffsync import Adapter
 from diffsync.exceptions import ObjectNotFound
 
-
 PEERINGDB_URL = "https://peeringdb.com/"
 PEERINGDB_API_KEY = os.environ.get("PEERINGDB_API_KEY", "").strip()
 
@@ -35,7 +36,7 @@ def load(self):
         if PEERINGDB_API_KEY:
             headers["Authorization"] = f"Api-Key {PEERINGDB_API_KEY}"
 
-        ix_data = requests.get(f"{PEERINGDB_URL}/api/ix/{self.ix_id}", headers=headers).json()
+        ix_data = requests.get(f"{PEERINGDB_URL}/api/ix/{self.ix_id}", headers=headers, timeout=60).json()
 
         for fac in ix_data["data"][0]["fac_set"]:
             # PeeringDB has no Region entity, so we must avoid duplicates