[pre-commit.ci] pre-commit autoupdate

.github/workflows/ci.yml

@@ -39,7 +39,7 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        python-version: ["3.7", "3.8", "3.9", "3.10"]
+        python-version: ["3.8", "3.9", "3.10", "3.11"]
         runs-on: [ubuntu-latest]
 
     steps:

.pre-commit-config.yaml

@@ -1,11 +1,11 @@
 repos:
   - repo: https://github.com/psf/black
-    rev: 22.3.0
+    rev: 23.11.0
     hooks:
       - id: black-jupyter
 
   - repo: https://github.com/pre-commit/pre-commit-hooks
-    rev: v4.3.0
+    rev: v4.5.0
     hooks:
       - id: check-added-large-files
       - id: check-case-conflict
@@ -19,7 +19,7 @@ repos:
       - id: trailing-whitespace
 
   - repo: https://github.com/pre-commit/pygrep-hooks
-    rev: v1.9.0
+    rev: v1.10.0
     hooks:
       - id: python-check-blanket-noqa
       - id: python-check-blanket-type-ignore
@@ -30,14 +30,14 @@ repos:
       - id: rst-inline-touching-normal
 
   - repo: https://github.com/pre-commit/mirrors-prettier
-    rev: "v2.7.1"
+    rev: "v3.1.0"
     hooks:
       - id: prettier
         types_or: [yaml, markdown, html, css, scss, javascript, json]
         args: [--prose-wrap=always]
 
   - repo: https://github.com/asottile/blacken-docs
-    rev: v1.12.1
+    rev: 1.16.0
     hooks:
       - id: blacken-docs
         additional_dependencies: [black==22.3.0]
@@ -49,26 +49,26 @@ repos:
         args: ["-a", "from __future__ import annotations"] # Python 3.7+
 
   - repo: https://github.com/asottile/pyupgrade
-    rev: v2.34.0
+    rev: v3.15.0
     hooks:
       - id: pyupgrade
         args: ["--py37-plus"]
 
   - repo: https://github.com/asottile/setup-cfg-fmt
-    rev: v1.20.1
+    rev: v2.5.0
     hooks:
       - id: setup-cfg-fmt
 
   - repo: https://github.com/hadialqattan/pycln
-    rev: v1.3.3
+    rev: v2.4.0
     hooks:
       - id: pycln
         additional_dependencies: [click<8.1]
         args: [--all]
         stages: [manual]
 
   - repo: https://github.com/asottile/yesqa
-    rev: v1.3.0
+    rev: v1.5.0
     hooks:
       - id: yesqa
         exclude: docs/conf.py
@@ -77,26 +77,26 @@ repos:
           - flake8-print
 
   - repo: https://github.com/pycqa/flake8
-    rev: 4.0.1
+    rev: 6.1.0
     hooks:
       - id: flake8
         exclude: docs/conf.py
         additional_dependencies: *flake8_dependencies
 
   - repo: https://github.com/pre-commit/mirrors-mypy
-    rev: v1.4.1
+    rev: v1.7.1
     hooks:
       - id: mypy
         files: src
         args: []
 
   - repo: https://github.com/codespell-project/codespell
-    rev: v2.1.0
+    rev: v2.2.6
     hooks:
       - id: codespell
 
   - repo: https://github.com/shellcheck-py/shellcheck-py
-    rev: v0.8.0.4
+    rev: v0.9.0.6
     hooks:
       - id: shellcheck
 
@@ -109,7 +109,7 @@ repos:
         exclude: .pre-commit-config.yaml
 
   - repo: https://github.com/mgedmin/check-manifest
-    rev: "0.48"
+    rev: "0.49"
     hooks:
       - id: check-manifest
         stages: [manual]

setup.cfg

@@ -9,7 +9,7 @@ author_email = [email protected]
 maintainer = CoffeaTeam admins
 maintainer_email = [email protected]
 license = BSD-3-Clause
-license_file = LICENSE
+license_files = LICENSE
 platforms =
     Any
 classifiers =
@@ -21,10 +21,6 @@ classifiers =
     Programming Language :: Python
     Programming Language :: Python :: 3
     Programming Language :: Python :: 3 :: Only
-    Programming Language :: Python :: 3.7
-    Programming Language :: Python :: 3.8
-    Programming Language :: Python :: 3.9
-    Programming Language :: Python :: 3.10
     Topic :: Scientific/Engineering
 project_urls =
     Documentation = https://fsspec_xrootd.readthedocs.io/
@@ -37,7 +33,7 @@ packages = find:
 install_requires =
     fsspec
     typing-extensions>=3.7;python_version<'3.8'
-python_requires = >=3.7
+python_requires = >=3.8
 include_package_data = True
 package_dir =
     =src
@@ -61,7 +57,7 @@ test =
     pytest>=6
 
 [flake8]
-extend-ignore = E203, E501, E722, B950
+extend-ignore = E203, E501, E722, B950, B905
 select = C,E,F,W,T,B,B9,I
 per-file-ignores =
     tests/*: T

src/fsspec_xrootd/xrootd.py

@@ -1,4 +1,4 @@
 from __future__ import annotations
 
 import asyncio
 import io
@@ -9,24 +9,27 @@
 from functools import partial
 from typing import Any, Callable, Iterable
 
-from fsspec.asyn import (  # type: ignore[import]
+from fsspec.asyn import (  # type: ignore[import-not-found]
     AsyncFileSystem,
     _run_coros_in_chunks,
     sync_wrapper,
 )
-from fsspec.spec import AbstractBufferedFile  # type: ignore[import]
-from XRootD import client  # type: ignore[import]
-from XRootD.client.flags import (  # type: ignore[import]
+from fsspec.spec import AbstractBufferedFile  # type: ignore[import-not-found]
+from XRootD import client  # type: ignore[import-not-found]
+from XRootD.client.flags import (  # type: ignore[import-not-found]
     DirListFlags,
     MkDirFlags,
     OpenFlags,
     QueryCode,
     StatInfoFlags,
 )
-from XRootD.client.responses import HostList, XRootDStatus  # type: ignore[import]
+from XRootD.client.responses import (  # type: ignore[import-not-found]
+    HostList,
+    XRootDStatus,
+)
 
 
 class ErrorCodes(IntEnum):
     INVALID_PATH = 400
 
 
@@ -34,7 +37,7 @@
     future: asyncio.Future[tuple[XRootDStatus, Any]],
     status: XRootDStatus,
     content: Any,
     servers: HostList,
 ) -> None:
     """Sets result of _async_wrap() future.
 
@@ -53,7 +56,7 @@
         return
     try:
         future.get_loop().call_soon_threadsafe(future.set_result, (status, content))
     except Exception as exc:
         future.get_loop().call_soon_threadsafe(future.set_exception, exc)
 
 
@@ -141,8 +144,7 @@
     return deets
 
 
 class XRootDFileSystem(AsyncFileSystem):  # type: ignore[misc]
-
     protocol = "root"
     root_marker = "/"
     default_timeout = 60
@@ -195,12 +197,12 @@
 
     @classmethod
     def _strip_protocol(cls, path: str | list[str]) -> Any:
-        if type(path) == str:
+        if isinstance(path, str):
             if path.startswith(cls.protocol):
                 return client.URL(path).path.rstrip("/") or cls.root_marker
             # assume already stripped
             return path.rstrip("/") or cls.root_marker
-        elif type(path) == list:
+        elif isinstance(path, list):
             return [cls._strip_protocol(item) for item in path]
         else:
             raise ValueError("Strip protocol not given string or list")
@@ -474,7 +476,7 @@
                 coros, batch_size=batch_size, nofiles=True
             )
             result_bufs = []
-            for (status, buffers) in results:
+            for status, buffers in results:
                 if not status.ok:
                     raise OSError(
                         f"File did not vector_read properly: {status.message}"
@@ -498,7 +500,6 @@
         batch_size: int | None = None,
         **kwargs: Any,
     ) -> list[bytes]:
-
         # TODO: on_error
         if max_gap is not None:
             # use utils.merge_offset_ranges
@@ -546,7 +547,6 @@
         cache_options: dict[Any, Any] | None = None,
         **kwargs: Any,
     ) -> XRootDFile:
-
         return XRootDFile(
             self,
             path,
@@ -599,8 +599,10 @@
                 **kwargs,
             )
             if compression is not None:
-                from fsspec.compression import compr  # type: ignore[import]
-                from fsspec.core import get_compression  # type: ignore[import]
+                from fsspec.compression import compr  # type: ignore[import-not-found]
+                from fsspec.core import (  # type: ignore[import-not-found]
+                    get_compression,
+                )
 
                 compression = get_compression(path, compression)
                 compress = compr[compression]