fix: update deps and safety

.pre-commit-config.yaml

@@ -1,17 +1,15 @@
 # See https://pre-commit.com for more information
 # See https://pre-commit.com/hooks.html for more hooks
 default_stages: [commit]
-default_language_version:
-  node: "16.16.0"
-  python: "python3.9"
+
 repos:
   - repo: https://github.com/MarcoGorelli/absolufy-imports
     rev: "v0.3.1"
     hooks:
       - id: absolufy-imports
 
   - repo: https://github.com/ambv/black
-    rev: "22.6.0"
+    rev: "24.8.0"
     hooks:
       - id: black
         language_version: python3.9
@@ -24,13 +22,13 @@ repos:
           - toml
 
   - repo: https://github.com/pycqa/isort
-    rev: 5.12.0
+    rev: 5.13.2
     hooks:
       - id: isort
         args: ["-m=VERTICAL_HANGING_INDENT", "--combine-as", "--profile=black"]
 
   - repo: https://github.com/pre-commit/pre-commit-hooks
-    rev: "v4.3.0"
+    rev: "v4.6.0"
     hooks:
       - id: trailing-whitespace
       - id: end-of-file-fixer
@@ -45,33 +43,38 @@ repos:
           - "--no-ensure-ascii"
 
   - repo: https://github.com/Lucas-C/pre-commit-hooks-safety
-    rev: "v1.3.0"
+    rev: "v1.3.3"
     hooks:
       - id: python-safety-dependencies-check
         files: pyproject.toml
-        args: [--disable-audit-and-monitor, -i=42835, -i=42837, -i=42836]
+        args: [--disable-audit-and-monitor]
 
   - repo: https://github.com/pre-commit/pygrep-hooks
-    rev: "v1.9.0"
+    rev: "v1.10.0"
     hooks:
       - id: python-no-log-warn
       - id: python-check-mock-methods
       - id: python-no-eval
 
   - repo: https://github.com/PyCQA/bandit
-    rev: "1.7.4"
+    rev: "1.7.9"
     hooks:
       - id: bandit
         files: "^admin_action_tools/.*"
 
   - repo: https://github.com/codespell-project/codespell
-    rev: "v2.1.0"
+    rev: "v2.3.0"
     hooks:
       - id: codespell
         args: ["-w"]
+        exclude: |
+            (?x)^(
+                admin_action_tools/tests/.*|
+                poetry.lock
+            )$
 
   - repo: https://github.com/alessandrojcm/commitlint-pre-commit-hook
-    rev: v9.0.0
+    rev: v9.18.0
     hooks:
       - id: commitlint
         stages: [commit-msg]

.safety-policy.yml

@@ -0,0 +1,65 @@
+# Safety Security and License Configuration file
+security:
+  ignore-vulnerabilities:
+    59293:
+      reason: we don't rely on it.
+    64976:
+      reason: we don't rely on it.
+    72110:
+      reason: we don't rely on it.
+    72109:
+      reason: we don't rely on it.
+    72111:
+      reason: we don't rely on it.
+    72095:
+      reason: we don't rely on it.
+    65771:
+      reason: we don't rely on it.
+    72519:
+      reason: we don't rely on it.
+    72520:
+      reason: we don't rely on it.
+    72515:
+      reason: we don't rely on it.
+    72521:
+      reason: we don't rely on it.
+    61586:
+      reason: we don't rely on it.
+    60956:
+      reason: we don't rely on it.
+    62126:
+      reason: we don't rely on it.
+    66742:
+      reason: fml, dev deps
+    64459:
+      reason: fml, dev deps
+    64396:
+      reason: fml, dev deps
+    67895:
+      reason: fml, dev deps
+    42837:
+      reason: fml, dev deps
+    42836:
+      reason: fml, dev deps
+    67136:
+      reason: fml, dev deps
+    61489:
+      reason: fml, dev deps
+    64436:
+      reason: fml, dev deps
+    64437:
+      reason: fml, dev deps
+    62156:
+      reason: fml, dev deps
+    58910:
+      reason: fml, dev deps
+    70716:
+      reason: fml, dev deps
+    70715:
+      reason: fml, dev deps
+    63073:
+      reason: fml, dev deps
+    58755:
+      reason: fml, dev deps
+    71064:
+      reason: fml, dev deps

CODE_OF_CONDUCT.md

@@ -5,7 +5,7 @@
 We as members, contributors, and leaders pledge to make participation in our
 community a harassment-free experience for everyone, regardless of age, body
 size, visible or invisible disability, ethnicity, sex characteristics, gender
-identity and expression, level of experience, education, socio-economic status,
+identity and expression, level of experience, education, socioeconomic status,
 nationality, personal appearance, race, religion, or sexual identity
 and orientation.
 

admin_action_tools/admin/confirm_tool.py

@@ -206,7 +206,7 @@ def _reconstruct_request_files():
                 log("Warning: no cached_object")
                 return
 
-            if type(cached_object) != self.model:
+            if not isinstance(cached_object, self.model):
                 # Do not use cache if the model doesn't match this model
                 log(f"Warning: cached_object is not of type {self.model}")
                 return

admin_action_tools/file_cache.py

@@ -27,6 +27,7 @@
 OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
 SOFTWARE.
 """
+
 from django.core.files.uploadedfile import InMemoryUploadedFile
 
 try:

admin_action_tools/tests/integration/confirm-tool/test_with_cache.py

@@ -2,6 +2,7 @@
 Tests confirmation of add/change
 on ModelAdmin that utilize caches
 """
+
 import os
 from importlib import reload
 

admin_action_tools/tests/integration/confirm-tool/test_with_change_actions.py

@@ -1,6 +1,7 @@
 """
 Tests confirmation of action that uses django-object-action
 """
+
 from importlib import reload
 
 from selenium.webdriver.common.by import By

admin_action_tools/tests/integration/confirm-tool/test_with_form_input_types.py

@@ -1,6 +1,7 @@
 """
 Tests with different form input types
 """
+
 from datetime import timedelta
 from importlib import reload
 

admin_action_tools/tests/integration/confirm-tool/test_with_inlines.py

@@ -4,6 +4,7 @@
 
 Does not test confirmation of inline changes
 """
+
 from importlib import reload
 
 import pkg_resources
@@ -112,7 +113,7 @@ def test_should_respect_get_inlines(self):
         # New in Django 3.0
         django_version = pkg_resources.get_distribution("Django").parsed_version
         if django_version.major < 3:
-            pytest.skip("get_inlines() introducted in Django 3.0, and is not in this version")
+            pytest.skip("get_inlines() introduced in Django 3.0, and is not in this version")
 
         shoppingmall_admin.ShoppingMallAdmin.inlines = []
         shoppingmall_admin.ShoppingMallAdmin.get_inlines = lambda self, request, obj=None: [

admin_action_tools/tests/integration/confirm-tool/test_with_s3_storage.py

@@ -3,6 +3,7 @@
 on ModelAdmin that utilize caches
 and S3 as a storage backend
 """
+
 import os
 from importlib import reload
 

admin_action_tools/tests/integration/form-tool/test_with_form_actions.py

@@ -1,6 +1,7 @@
 """
 Tests confirmation of action that uses django-object-action
 """
+
 from importlib import reload
 
 from selenium.webdriver.common.by import By

admin_action_tools/tests/integration/form-tool/test_with_multiple_forms.py

@@ -1,6 +1,7 @@
 """
 Tests confirmation of action that uses django-object-action
 """
+
 from importlib import reload
 
 from selenium.webdriver.common.by import By

admin_action_tools/tests/unit/confirm-tool/test_confirmation_using_file_cache.py

@@ -10,6 +10,7 @@
 This is arguably the most we fiddle with the Django request
 Thus we should test it extensively
 """
+
 import time
 from unittest import mock