Merge pull request #1975 from braingram/update_pre-commit

Update pre commit

Author: braingram

.pre-commit-config.yaml

@@ -2,7 +2,7 @@ exclude: "asdf/(_extern||_jsonschema)/.*"
 repos:
 
 - repo: https://github.com/pre-commit/pre-commit-hooks
-  rev: v5.0.0
+  rev: v6.0.0
   hooks:
     - id: check-added-large-files
     - id: check-ast
@@ -37,13 +37,13 @@ repos:
         - tomli
 
 - repo: https://github.com/ikamensh/flynt/
-  rev: '1.0.1'
+  rev: '1.0.6'
   hooks:
     - id: flynt
       exclude: "asdf/(extern||_jsonschema)/.*"
 
 - repo: https://github.com/astral-sh/ruff-pre-commit
-  rev: 'v0.11.6'
+  rev: 'v0.13.0'
   hooks:
     - id: ruff
       args: ["--fix"]
@@ -54,7 +54,7 @@ repos:
     - id: black
 
 - repo: https://github.com/asottile/blacken-docs
-  rev: '1.19.1'
+  rev: '1.20.0'
   hooks:
     - id: blacken-docs
 

asdf/_block/external.py

@@ -65,6 +65,6 @@ def clear(self):
 def relative_uri_for_index(uri, index):
     # get the os-native separated path for this uri
     path = util._patched_urllib_parse.urlparse(uri).path
-    dirname, filename = os.path.split(path)
+    _, filename = os.path.split(path)
     filename = os.path.splitext(filename)[0] + f"{index:04d}.asdf"
     return filename

asdf/_commands/exploded.py

@@ -67,7 +67,7 @@ def implode(input_, output=None, resolve_references=False):
         If `True` resolve all external references before saving.
     """
     if output is None:
-        base, ext = os.path.splitext(input_)
+        base, _ = os.path.splitext(input_)
         output = base + "_all" + ".asdf"
     with asdf.open(input_) as ff:
         ff2 = AsdfFile(ff)
@@ -121,7 +121,7 @@ def explode(input_, output=None):
         The output file.
     """
     if output is None:
-        base, ext = os.path.splitext(input_)
+        base, _ = os.path.splitext(input_)
         output = base + "_exploded" + ".asdf"
     with asdf.open(input_) as ff:
         ff.write_to(output, all_array_storage="external")

asdf/_commands/main.py

@@ -50,7 +50,7 @@ def help_(args):
 
 
 def main_from_args(args):
-    parser, subparsers = make_argparser()
+    parser, _ = make_argparser()
 
     args = parser.parse_args(args)
 

asdf/_node_info.py

@@ -361,7 +361,7 @@ def from_root_node(cls, key, root_identifier, root_node, schema=None, extension_
                 # node is the item in the tree
                 # We might sometimes not want to use that node directly
                 # but instead using a different node for traversal.
-                t_node, traversable, from_converter = _make_traversable(node, extension_manager)
+                t_node, traversable, _ = _make_traversable(node, extension_manager)
                 if (is_container(node) or traversable) and id(node) in seen:
                     info = NodeSchemaInfo(
                         key,

asdf/_tests/_block/test_io.py

@@ -169,7 +169,7 @@ def test_write_oversized_block():
     data = np.ones(30, dtype="uint8")
     raw_fd = io.BytesIO()
     fd = generic_io.get_file(raw_fd, mode="rw")
-    with pytest.raises(RuntimeError, match="Block used size.*"):
+    with pytest.raises(RuntimeError, match=r"Block used size.*"):
         bio.write_block(fd, data, allocated_size=0)
     assert fd.tell() == 0
 
@@ -190,7 +190,7 @@ def test_fd_not_seekable():
 
     np.testing.assert_array_equal(d, data)
 
-    with pytest.raises(ValueError, match="write_block received offset.*"):
+    with pytest.raises(ValueError, match=r"write_block received offset.*"):
         bio.write_block(fd, data, offset=0)
 
 
@@ -223,14 +223,14 @@ def test_read_from_closed(tmp_path):
         bio.write_block(fd, data, stream=True)
     with generic_io.get_file(fn, mode="rw") as fd:
         _, _, _, callback = bio.read_block(fd, offset=0, lazy_load=True)
-    with pytest.raises(OSError, match="ASDF file has already been closed. Can not get the data."):
+    with pytest.raises(OSError, match=r"ASDF file has already been closed\. Can not get the data\."):
         callback()
 
 
 @pytest.mark.parametrize("data", [np.ones(10, dtype="f4"), np.ones((3, 3), dtype="uint8")])
 def test_invalid_data(data):
     fd = generic_io.get_file(io.BytesIO(), mode="rw")
-    with pytest.raises(ValueError, match="Data must be of.*"):
+    with pytest.raises(ValueError, match=r"Data must be of.*"):
         bio.write_block(fd, data, stream=True)
 
 
@@ -337,7 +337,7 @@ def test_read_block_index_no_header(tmp_path):
     generate_block_index_file(fn, values=values, offset=0)
     with generic_io.get_file(fn, "r") as fd:
         fd.seek(len(constants.INDEX_HEADER))
-        with pytest.raises(BlockIndexError, match="Failed to read block index.*"):
+        with pytest.raises(BlockIndexError, match=r"Failed to read block index.*"):
             assert bio.read_block_index(fd) == values
 
 

asdf/_tests/_block/test_options.py

@@ -68,27 +68,27 @@ def test_default_compression():
 
 @pytest.mark.parametrize("invalid_storage", invalid_storage_types)
 def test_invalid_storage_type_init(invalid_storage):
-    with pytest.raises(ValueError, match="array_storage must be one of.*"):
+    with pytest.raises(ValueError, match=r"array_storage must be one of.*"):
         Options(invalid_storage)
 
 
 @pytest.mark.parametrize("invalid_storage", invalid_storage_types)
 def test_invalid_storage_attr(invalid_storage):
     o = Options("internal")
-    with pytest.raises(ValueError, match="array_storage must be one of.*"):
+    with pytest.raises(ValueError, match=r"array_storage must be one of.*"):
         o.storage_type = invalid_storage
 
 
 @pytest.mark.parametrize("invalid_compression", invalid_compression_types)
 def test_invalid_compression_attr(invalid_compression):
     o = Options("internal")
-    with pytest.raises(ValueError, match="Invalid compression.*"):
+    with pytest.raises(ValueError, match=r"Invalid compression.*"):
         o.compression = invalid_compression
 
 
 @pytest.mark.parametrize("invalid_compression", invalid_compression_types)
 def test_invalid_compression_init(invalid_compression):
-    with pytest.raises(ValueError, match="Invalid compression.*"):
+    with pytest.raises(ValueError, match=r"Invalid compression.*"):
         Options("internal", invalid_compression)
 
 

asdf/_tests/_block/test_reader.py

@@ -140,7 +140,7 @@ def test_invalid_block_index(tmp_path, invalid_block_index):
             with pytest.warns(AsdfBlockIndexWarning, match="Failed to read block index"):
                 check(read_blocks(fd, lazy_load=True))
         else:
-            with pytest.raises(ValueError, match="Header size.*"):
+            with pytest.raises(ValueError, match=r"Header size.*"):
                 check(read_blocks(fd, lazy_load=True))
 
 
@@ -168,7 +168,7 @@ def test_invalid_block_in_index_with_valid_magic(tmp_path):
 
 def test_closed_file(tmp_path):
     fn = tmp_path / "test.bin"
-    with gen_blocks(fn=fn, with_index=True) as (fd, check):
+    with gen_blocks(fn=fn, with_index=True) as (fd, _):
         blocks = read_blocks(fd, lazy_load=True)
         blk = blocks[1]
     with pytest.raises(OSError, match="Attempt to load block from closed file"):
@@ -190,7 +190,7 @@ def test_bad_checksum(validate_checksums):
 
     with generic_io.get_file(buff, mode="r") as fd:
         if validate_checksums:
-            with pytest.raises(ValueError, match=".* does not match given checksum"):
+            with pytest.raises(ValueError, match=r".* does not match given checksum"):
                 read_blocks(fd, lazy_load=False, validate_checksums=validate_checksums)[0].data
         else:
             read_blocks(fd, lazy_load=False, validate_checksums=validate_checksums)[0].data

asdf/_tests/_regtests/test_1539.py

@@ -17,6 +17,6 @@ def test_invalid_seek_and_read_from_closed_memoryio():
     b.seek(0)
     f = asdf.generic_io.get_file(b)
     f.close()
-    with pytest.raises(IOError, match="I/O operation on closed file."):
+    with pytest.raises(IOError, match=r"I/O operation on closed file\."):
         f.read_into_array(10)
     assert b.tell() == 0

asdf/_tests/test_extension.py

@@ -1037,7 +1037,7 @@ class MailboxExtension:
         assert "mailbox" not in sys.modules
 
         # as the module hasn't been loaded, the converter shouldn't be found
-        with pytest.raises(KeyError, match="No support available for Python type 'mailbox.Mailbox'"):
+        with pytest.raises(KeyError, match=r"No support available for Python type 'mailbox\.Mailbox'"):
             extension_manager.get_converter_for_type(typ)
 
         # make sure inspecting the type didn't import the module