Merge pull request #265 from beeware/autoupdates/pre-commit

Auto-update `pre-commit` hooks

Author: freakboy3742

.pre-commit-config.yaml

@@ -1,6 +1,6 @@
 repos:
   - repo: https://github.com/pre-commit/pre-commit-hooks
-    rev: v4.3.0
+    rev: v4.4.0
     hooks:
       - id: check-toml
       - id: check-yaml
@@ -9,29 +9,20 @@ repos:
       - id: end-of-file-fixer
       - id: trailing-whitespace
   - repo: https://github.com/PyCQA/isort
-    # For split_on_trailing_comma. Should be in the release after 5.10.1
-    rev: 12cc5fbd67eebf92eb2213b03c07b138ae1fb448
+    rev: 5.12.0
     hooks:
       - id: isort
         additional_dependencies: [toml]
   - repo: https://github.com/asottile/pyupgrade
-    rev: v2.38.2
+    rev: v3.3.1
     hooks:
       - id: pyupgrade
         args: [--py36-plus]
-  # Docformatter is RST-unaware, and may break markup by inserting or removing newlines
-  # (see https://github.com/PyCQA/docformatter/issues/124 and linked issues). Until this
-  # is fixed, it's not safe to run it automatically.
-  # - repo: https://github.com/myint/docformatter
-  #   rev: v1.5.0
-  #   hooks:
-  #   - id: docformatter
-  #     args: [--in-place]
   - repo: https://github.com/psf/black
-    rev: 22.8.0
+    rev: 23.1.0
     hooks:
     - id: black
   - repo: https://github.com/PyCQA/flake8
-    rev: 5.0.4
+    rev: 6.0.0
     hooks:
       - id: flake8

changes/265.misc.rst

@@ -0,0 +1 @@
+Updated ``pre-commit`` hooks to the latest version.

src/rubicon/objc/api.py

@@ -482,7 +482,6 @@ def _get_property_attributes(self):
         return (objc_property_attribute_t * len(attrs))(*attrs)
 
     def class_register(self, class_ptr, attr_name):
-
         ivar_name = "_" + attr_name
 
         add_ivar(class_ptr, ivar_name, self.vartype)
@@ -519,7 +518,6 @@ def _objc_getter(objc_self, _cmd):
             return value
 
         def _objc_setter(objc_self, _cmd, new_value):
-
             if self._is_py_object and self.weak:
                 # Don't store the object itself but only a Python weakref.
                 new_value = weakref.ref(new_value)
@@ -575,7 +573,6 @@ def _objc_setter(objc_self, _cmd, new_value):
         libobjc.class_addProperty(class_ptr, ensure_bytes(attr_name), attrs, len(attrs))
 
     def dealloc_callback(self, objc_self, attr_name):
-
         ivar_name = "_" + attr_name
 
         if self._ivar_weak:
@@ -1286,7 +1283,6 @@ def _new_from_class_statement(cls, name, bases, attrs, *, protocols):
         user_dealloc = attrs.get("dealloc", None)
 
         def _new_delloc(objc_self, _cmd):
-
             # Invoke user-defined dealloc.
             if user_dealloc:
                 user_dealloc(objc_self, _cmd)
@@ -2000,7 +1996,6 @@ def __subclasscheck__(self, subclass):
 except NameError:
 
     class WrappedPyObject(NSObject):
-
         wrapped_pointer = objc_ivar(c_void_p)
 
         @objc_rawmethod

tests/test_core.py

@@ -1107,7 +1107,6 @@ def test_class_properties(self):
         NSURL = ObjCClass("NSURL")
 
         class URLBox(NSObject):
-
             url = objc_property(ObjCInstance)
             data = objc_property(ObjCInstance)
 
@@ -1242,7 +1241,6 @@ class NonObjectProperties(NSObject):
         self.assertEqual(r.size.height, 78)
 
     def test_class_nonobject_properties_weak(self):
-
         with self.assertRaises(TypeError):
 
             class WeakNonObjectProperties(NSObject):
@@ -1253,7 +1251,6 @@ class StrongObjectProperties(NSObject):
             object = objc_property(ObjCInstance)
 
         with autoreleasepool():
-
             properties = StrongObjectProperties.alloc().init()
 
             obj = NSObject.alloc().init()
@@ -1272,7 +1269,6 @@ class WeakObjectProperties(NSObject):
             object = objc_property(ObjCInstance, weak=True)
 
         with autoreleasepool():
-
             properties = WeakObjectProperties.alloc().init()
 
             obj = NSObject.alloc().init()
@@ -1527,7 +1523,6 @@ def test_objcinstance_python_attribute_freed(self):
         released."""
 
         with autoreleasepool():
-
             obj = NSObject.alloc().init()
 
             # Use a custom object as attribute value so that we can keep a weak reference.
@@ -1558,7 +1553,6 @@ class TestO:
         self.assertIsNone(wr_python_object())
 
     def test_objcinstance_release_owned(self):
-
         # Create an object which we own.
         obj = NSObject.alloc().init()
 
@@ -1576,7 +1570,6 @@ def test_objcinstance_release_owned(self):
         gc.collect()
 
     def test_objcinstance_autorelease_owned(self):
-
         # Create an object which we own.
         obj = NSObject.alloc().init()