Bump dependency versions in the pre-commit hooks and address warnings

and errors in the code base that were introduced from the version
updates.

Author: Ryan Whitmore

.pre-commit-config.yaml

@@ -5,39 +5,37 @@ exclude: |
   )$
 repos:
   - repo: https://github.com/pre-commit/pre-commit-hooks
-    rev: v4.4.0
+    rev: v4.5.0
     hooks:
       - id: debug-statements
       - id: check-merge-conflict
   - repo: https://github.com/psf/black
-    rev: 23.3.0
+    rev: 23.12.1
     hooks:
       - id: black
         language_version: python3
   - repo: https://github.com/pycqa/flake8
-    rev: 6.0.0
+    rev: 7.0.0
     hooks:
       - id: flake8
   - repo: https://github.com/pycqa/isort
-    rev: 5.12.0
+    rev: 5.13.2
     hooks:
       - id: isort
-  - repo: https://github.com/humitos/mirrors-autoflake.git
-    rev: v1.1
+  - repo: https://github.com/PyCQA/autoflake
+    rev: v2.2.1
     hooks:
       - id: autoflake
         exclude: |
           (?x)^(
               .*/?__init__\.py|
           )$
         args:
-          [
-            "--in-place",
-            "--remove-all-unused-imports",
-            "--remove-unused-variable",
-          ]
+          - --in-place
+          - --remove-all-unused-imports
+          - --remove-unused-variables
   - repo: https://github.com/pre-commit/mirrors-mypy
-    rev: v1.4.0
+    rev: v1.8.0
     hooks:
       - id: mypy
         additional_dependencies:

etuples/core.py

@@ -102,7 +102,7 @@ def _repr_pretty_(self, p, cycle):
 
     def __eq__(self, other):
         return (
-            type(self) == type(other)
+            type(self) is type(other)
             and self.arg == other.arg
             and self.value == other.value
         )
@@ -132,7 +132,7 @@ def __new__(cls, seq=None, **kwargs):
         # It does, however, remove it for the check below.
         kwargs.pop("evaled_obj", None)
 
-        if seq is not None and not kwargs and type(seq) == cls:
+        if seq is not None and not kwargs and type(seq) is cls:
             return seq
 
         res = super().__new__(cls)

tests/test_core.py

@@ -98,7 +98,7 @@ def test_op(*args):
 
     e1_obj = e1.evaled_obj
     assert len(e1_obj) == 3
-    assert all(type(o) == object for o in e1_obj)
+    assert all(type(o) is object for o in e1_obj)
 
     # Make sure we don't re-create the cached `evaled_obj`
     e1_obj_2 = e1.evaled_obj
@@ -109,17 +109,17 @@ def test_op(*args):
 
     # Make sure we didn't convert this single tuple value to
     # an `etuple`
-    assert type(e2[1]) == tuple
+    assert type(e2[1]) is tuple
 
     # Slices should be `etuple`s, though.
     assert isinstance(e2[:1], ExpressionTuple)
     assert e2[1] == e2[1:2][0]
 
     e2_obj = e2.evaled_obj
 
-    assert type(e2_obj) == tuple
+    assert type(e2_obj) is tuple
     assert len(e2_obj) == 4
-    assert all(type(o) == object for o in e2_obj)
+    assert all(type(o) is object for o in e2_obj)
     # Make sure that it used `e1`'s original `evaled_obj`
     assert e2_obj[1:] == e1_obj
 

tests/test_dispatch.py

@@ -15,7 +15,7 @@ def __init__(self, rator, rands):
 
     def __eq__(self, other):
         return (
-            type(self) == type(other)
+            type(self) is type(other)
             and self.rator == other.rator
             and self.rands == other.rands
         )