Enable isort rules (#279)

Author: nathanjmcdougall

pins/_types.py

@@ -1,4 +1,4 @@
-from typing import Union
 from io import IOBase
+from typing import Union
 
 StrOrFile = Union[str, IOBase]

pins/boards.py

@@ -1,25 +1,23 @@
-import logging
-import tempfile
-import shutil
+import functools
 import inspect
+import logging
 import re
-import functools
-
+import shutil
+import tempfile
+from datetime import datetime, timedelta
 from io import IOBase
 from pathlib import Path
-from importlib_resources import files
-from datetime import datetime, timedelta
+from typing import Mapping, Optional, Protocol, Sequence
 
-from typing import Protocol, Sequence, Optional, Mapping
+from importlib_resources import files
 
-from .versions import VersionRaw, guess_version, version_setup
-from .meta import Meta, MetaRaw, MetaFactory
-from .errors import PinsError, PinsVersionError
-from .drivers import load_data, save_data, load_file, default_title
-from .utils import inform, warn_deprecated, ExtendMethodDoc
-from .config import get_allow_rsc_short_name
 from .cache import PinsCache
-
+from .config import get_allow_rsc_short_name
+from .drivers import default_title, load_data, load_file, save_data
+from .errors import PinsError, PinsVersionError
+from .meta import Meta, MetaFactory, MetaRaw
+from .utils import ExtendMethodDoc, inform, warn_deprecated
+from .versions import VersionRaw, guess_version, version_setup
 
 _log = logging.getLogger(__name__)
 
@@ -1164,9 +1162,10 @@ def prepare_pin_version(self, pin_dir_path, x, name: "str | None", *args, **kwar
         # TODO: move out data_preview logic? Can we draw some limits here?
         #       note that the R library uses jsonlite::toJSON
 
-        import pandas as pd
         import json
 
+        import pandas as pd
+
         if isinstance(x, pd.DataFrame):
             # TODO(compat) is 100 hard-coded?
             # Note that we go df -> json -> dict, to take advantage of pandas type conversions

pins/cache.py

@@ -1,16 +1,16 @@
-import humanize
 import logging
 import os
-import time
 import shutil
+import time
 import urllib.parse
+from pathlib import Path
 
-from fsspec.implementations.cached import SimpleCacheFileSystem
+import humanize
 from fsspec import register_implementation
-from pathlib import Path
+from fsspec.implementations.cached import SimpleCacheFileSystem
 
 from .config import get_cache_dir
-from .utils import inform, hash_name
+from .utils import hash_name, inform
 
 _log = logging.getLogger(__name__)
 

pins/config.py

@@ -1,8 +1,8 @@
-import appdirs
 import os
-
 from types import SimpleNamespace
 
+import appdirs
+
 PINS_NAME = "pins-py"
 PINS_ENV_DATA_DIR = "PINS_DATA_DIR"
 PINS_ENV_CACHE_DIR = "PINS_CACHE_DIR"

pins/constructors.py

@@ -1,11 +1,12 @@
-import fsspec
 import os
 import tempfile
 import warnings
 
-from .boards import BaseBoard, BoardRsConnect, BoardManual, board_deparse
-from .cache import PinsCache, PinsRscCacheMapper, PinsAccessTimeCache, prefix_cache
-from .config import get_data_dir, get_cache_dir
+import fsspec
+
+from .boards import BaseBoard, BoardManual, BoardRsConnect, board_deparse
+from .cache import PinsAccessTimeCache, PinsCache, PinsRscCacheMapper, prefix_cache
+from .config import get_cache_dir, get_data_dir
 
 # Kept here for backward-compatibility reasons.
 board_deparse  # Note that this is not a constructor, but a function to represent them.

pins/drivers.py

@@ -1,10 +1,9 @@
 from pathlib import Path
+from typing import Sequence
 
-from .config import get_allow_pickle_read, PINS_ENV_INSECURE_READ
-from .meta import Meta
+from .config import PINS_ENV_INSECURE_READ, get_allow_pickle_read
 from .errors import PinsInsecureReadError
-
-from typing import Sequence
+from .meta import Meta
 
 # TODO: move IFileSystem out of boards, to fix circular import
 # from .boards import IFileSystem

pins/meta.py

@@ -1,13 +1,11 @@
-from typing import ClassVar
-from dataclasses import dataclass, asdict, field, fields, InitVar
+from dataclasses import InitVar, asdict, dataclass, field, fields
 from pathlib import Path
+from typing import ClassVar, List, Mapping, Optional, Sequence, Union
 
 import yaml
 
-from typing import Mapping, Union, Sequence, Optional, List
-
-from .versions import VersionRaw, Version, guess_version
-from ._types import StrOrFile, IOBase
+from ._types import IOBase, StrOrFile
+from .versions import Version, VersionRaw, guess_version
 
 META_FILENAME = "data.txt"
 DEFAULT_API_VERSION = 1

pins/rsconnect/api.py

@@ -1,17 +1,15 @@
 import logging
 import os
-import requests
 import tempfile
-
+from collections.abc import Mapping
 from dataclasses import dataclass
-from pathlib import Path
 from functools import partial
 from io import IOBase
+from pathlib import Path
+from typing import Generic, Sequence, TypeVar
 from urllib.parse import urlencode
 
-from collections.abc import Mapping
-from typing import Sequence, TypeVar, Generic
-
+import requests
 
 RSC_API_KEY = "CONNECT_API_KEY"
 RSC_CODE_OBJECT_DOES_NOT_EXIST = 4

pins/rsconnect/fs.py

@@ -1,23 +1,22 @@
-from dataclasses import dataclass, asdict, field, fields
+from dataclasses import asdict, dataclass, field, fields
 from pathlib import Path
+from typing import Sequence
 
 from fsspec import AbstractFileSystem
 
-from typing import Sequence
-
+from ..utils import isfilelike
 from .api import (
-    RsConnectApi,
+    RSC_CODE_OBJECT_DOES_NOT_EXIST,
     BaseEntity,
-    User,
-    Content,
     Bundle,
-    RsConnectApiMissingContentError,
+    Content,
+    RsConnectApi,
     RsConnectApiError,
-    RsConnectApiResultError,
+    RsConnectApiMissingContentError,
     RsConnectApiRequestError,
-    RSC_CODE_OBJECT_DOES_NOT_EXIST,
+    RsConnectApiResultError,
+    User,
 )
-from ..utils import isfilelike
 
 # Misc ----
 

pins/tests/conftest.py

@@ -1,10 +1,11 @@
-import pytest
-import tempfile
 import os
+import tempfile
+from pathlib import Path
 
+import pytest
 from importlib_resources import files
 from pytest import mark as m
-from pathlib import Path
+
 from pins.tests.helpers import BoardBuilder, RscBoardBuilder, Snapshot, rm_env
 
 EXAMPLE_REL_PATH = "pins/tests/pins-compat"