Fix GH actions

.github/workflows/test.yml

@@ -11,22 +11,29 @@ on:
 jobs:
   code-check:
     name: Code checking
-    runs-on: macos-latest
+    runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v3
+      - uses: actions/checkout@v4
       - name: Set up Python
         uses: actions/setup-python@v4
         with:
           python-version: "3.11"
       - name: Install poetry
         run: pipx install poetry
+      - name: Set python version for poetry
+        run: poetry env use python3.11
       - name: Install dependencies
         run: poetry install
+      - name: Run Isort
+        run: poetry run -v -- isort -c notifications/ sample_website/
+      - name: Run Black
+        run: poetry run -v -- black --check notifications/ sample_website/
       - name: Run Pylint
-        run: poetry run -v -- pylint --rcfile=pyproject.toml notifications/ sample_website/
+        run: poetry run -v -- pylint --rcfile=pyproject.toml --recursive=y -v notifications/ sample_website/
       - name: Run Bandit
         run: poetry run -v -- bandit -c pyproject.toml -r notifications/ sample_website/
-
+      - name: Run Mypy
+        run: poetry run -v -- mypy
 
   test:
     name: Testing
@@ -37,23 +44,25 @@ jobs:
         python-version: ["3.8", "3.9", "3.10", "3.11"]
 
     steps:
-      - uses: actions/checkout@v3
+      - uses: actions/checkout@v4
       - name: Set up Python ${{ matrix.python-version }}
         uses: actions/setup-python@v4
         with:
           python-version: ${{ matrix.python-version }}
       - name: Install poetry
         run: pipx install poetry
+      - name: Set python version for poetry
+        run: poetry env use python${{ matrix.python-version }}
       - name: Install dependencies
         run: poetry install
       - name: Run tests
-        run: poetry run pytest
+        run: poetry run -- pytest
         env:
           COVERAGE_FILE: ".coverage.${{ matrix.python_version }}"
       - name: Store coverage file
-        uses: actions/upload-artifact@v3
+        uses: actions/upload-artifact@v4
         with:
-          name: coverage
+          name: coverage-${{ matrix.python_version }}
           path: .coverage.${{ matrix.python_version }}
 
 
@@ -65,12 +74,13 @@ jobs:
       pull-requests: write
       contents: write
     steps:
-      - uses: actions/checkout@v3
+      - uses: actions/checkout@v4
 
-      - uses: actions/download-artifact@v3
+      - uses: actions/download-artifact@v4
         id: download
         with:
-          name: 'coverage'
+          pattern: coverage-*
+          merge-multiple: true
 
       - name: Coverage comment
         id: coverage_comment
@@ -80,7 +90,7 @@ jobs:
           MERGE_COVERAGE_FILES: true
 
       - name: Store Pull Request comment to be posted
-        uses: actions/upload-artifact@v3
+        uses: actions/upload-artifact@v4
         if: steps.coverage_comment.outputs.COMMENT_FILE_WRITTEN == 'true'
         with:
           name: python-coverage-comment-action

.pre-commit-config.yaml

@@ -21,7 +21,7 @@ repos:
     hooks:
     -   id: pylint
         name: pylint
-        entry: poetry run pylint
+        entry: poetry run -- pylint
         language: system
         args: ["--rcfile=pyproject.toml"]
 

dockerfile

@@ -20,4 +20,4 @@ RUN pyenv local system 3.8 3.9 3.10
 RUN apt-get --purge autoremove -y gnupg; \
     rm -rf /var/cache/apt/lists;
 
-ENTRYPOINT poetry install && poetry run pre-commit install && /bin/bash
+ENTRYPOINT poetry install && poetry run -- pre-commit install && /bin/bash

notifications/admin.py

@@ -28,9 +28,9 @@ def get_queryset(self, request: HttpRequest):
         return qs.prefetch_related("actor")
 
     @admin.action(description=gettext_lazy("Mark selected notifications as unread"))
-    def mark_unread(self, request: HttpRequest, queryset: NotificationQuerySet):  # pylint: disable=unused-argument
+    def mark_unread(self, request: HttpRequest, queryset: NotificationQuerySet):
         queryset.update(unread=True)
 
     @admin.action(description=gettext_lazy("Mark selected notifications as read"))
-    def mark_read(self, request: HttpRequest, queryset: NotificationQuerySet):  # pylint: disable=unused-argument
+    def mark_read(self, request: HttpRequest, queryset: NotificationQuerySet):
         queryset.update(unread=False)

notifications/models/base.py

@@ -1,5 +1,6 @@
 # -*- coding: utf-8 -*-
 import datetime
+from typing import Union
 
 from django.conf import settings
 from django.contrib.contenttypes.fields import GenericForeignKey
@@ -130,7 +131,7 @@ def __str__(self):
             return _("%(actor)s %(verb)s %(action_object)s %(timesince)s ago") % ctx
         return _("%(actor)s %(verb)s %(timesince)s ago") % ctx
 
-    def timesince(self, now: None | datetime.datetime = None) -> str:
+    def timesince(self, now: Union[None, datetime.datetime] = None) -> str:
         """
         Shortcut for the ``django.utils.timesince.timesince`` function of the
         current timestamp.

notifications/querysets.py

@@ -1,4 +1,4 @@
-from typing import Type
+from typing import Type, Union
 
 from django.core.exceptions import ImproperlyConfigured
 from django.db import models
@@ -46,7 +46,7 @@ def read(self, include_deleted: bool = False) -> "NotificationQuerySet":
         # in this case, to improve query performance, don't filter by 'deleted' field
         return self.filter(unread=False)
 
-    def mark_all_as_read(self, recipient: None | Type[AbstractUser] = None) -> int:
+    def mark_all_as_read(self, recipient: Union[None, Type[AbstractUser]] = None) -> int:
         """Mark as read any unread messages in the current queryset.
 
         Optionally, filter these by recipient first.
@@ -59,7 +59,7 @@ def mark_all_as_read(self, recipient: None | Type[AbstractUser] = None) -> int:
 
         return qset.update(unread=False)
 
-    def mark_all_as_unread(self, recipient: None | Type[AbstractUser] = None) -> int:
+    def mark_all_as_unread(self, recipient: Union[None, Type[AbstractUser]] = None) -> int:
         """Mark as unread any read messages in the current queryset.
 
         Optionally, filter these by recipient first.
@@ -81,7 +81,7 @@ def active(self) -> "NotificationQuerySet":
         assert_soft_delete()
         return self.filter(deleted=False)
 
-    def mark_all_as_deleted(self, recipient: None | Type[AbstractUser] = None) -> int:
+    def mark_all_as_deleted(self, recipient: Union[None, Type[AbstractUser]] = None) -> int:
         """Mark current queryset as deleted.
         Optionally, filter by recipient first.
         """
@@ -92,7 +92,7 @@ def mark_all_as_deleted(self, recipient: None | Type[AbstractUser] = None) -> in
 
         return qset.update(deleted=True)
 
-    def mark_all_as_active(self, recipient: None | Type[AbstractUser] = None) -> int:
+    def mark_all_as_active(self, recipient: Union[None, Type[AbstractUser]] = None) -> int:
         """Mark current queryset as active(un-deleted).
         Optionally, filter by recipient first.
         """
@@ -103,13 +103,13 @@ def mark_all_as_active(self, recipient: None | Type[AbstractUser] = None) -> int
 
         return qset.update(deleted=False)
 
-    def mark_as_unsent(self, recipient: None | Type[AbstractUser] = None) -> int:
+    def mark_as_unsent(self, recipient: Union[None, Type[AbstractUser]] = None) -> int:
         qset = self.sent()
         if recipient:
             qset = qset.filter(recipient=recipient)
         return qset.update(emailed=False)
 
-    def mark_as_sent(self, recipient: None | Type[AbstractUser] = None) -> int:
+    def mark_as_sent(self, recipient: Union[None, Type[AbstractUser]] = None) -> int:
         qset = self.unsent()
         if recipient:
             qset = qset.filter(recipient=recipient)

notifications/settings.py

@@ -1,6 +1,6 @@
 """ Django notifications settings file """
 # -*- coding: utf-8 -*-
-from typing import Any, TypedDict
+from typing import Any, TypedDict, Union
 
 from django.conf import settings
 
@@ -39,15 +39,17 @@ class NotificationSettings:
     apps, and test helpers like `override_settings` may not work as expected.
     """
 
-    def __init__(self, defaults: None | NotificationDefaultsType = None):
-        self._user_settings = None
-        self.defaults = defaults or NotificationDefaultsType(**NOTIFICATION_DEFAULTS)
+    def __init__(self, defaults: Union[None, NotificationDefaultsType] = None):
+        self._user_settings: Union[NotificationDefaultsType, None] = None
+        self.defaults = defaults or NotificationDefaultsType(**NOTIFICATION_DEFAULTS)  # type: ignore[typeddict-item]
         self._cached_attrs: set[str] = set()
 
     @property
     def user_settings(self) -> NotificationDefaultsType:
         if not self._user_settings:
-            self._user_settings = NotificationDefaultsType(**getattr(settings, "DJANGO_NOTIFICATIONS_CONFIG") or {})
+            self._user_settings = NotificationDefaultsType(
+                **getattr(settings, "DJANGO_NOTIFICATIONS_CONFIG") or {}
+            )  # type: ignore[typeddict-item]
         return self._user_settings
 
     def __getattr__(self, attr: str) -> NotificationDefaultsType:
@@ -56,10 +58,10 @@ def __getattr__(self, attr: str) -> NotificationDefaultsType:
 
         try:
             # Check if present in user settings
-            val = self.user_settings[attr]
+            val = self.user_settings[attr]  # type: ignore[literal-required]
         except KeyError:
             # Fall back to defaults
-            val = self.defaults[attr]
+            val = self.defaults[attr]  # type: ignore[literal-required]
 
         # Cache the result
         self._cached_attrs.add(attr)
@@ -73,7 +75,9 @@ def reload(self):
         self._user_settings = None
 
 
-notification_settings = NotificationSettings(NOTIFICATION_DEFAULTS)
+notification_settings = NotificationSettings(
+    NotificationDefaultsType(**NOTIFICATION_DEFAULTS)  # type: ignore[typeddict-item]
+)
 
 
 def reload_notification_settings(*args: Any, **kwargs: Any):  # pylint: disable=unused-argument

notifications/tests/tests.py

@@ -4,7 +4,6 @@
 Replace this with more appropriate tests for your application.
 """
 # -*- coding: utf-8 -*-
-# pylint: disable=too-many-lines,missing-docstring
 import json
 from datetime import datetime, timezone
 
@@ -107,7 +106,7 @@ def test_notify_send_return_val(self):
                 # only check types for now
                 self.assertEqual(type(result[1][0]), Notification)
 
-    def test_notify_send_return_val_group(self):  # pylint: disable=invalid-name
+    def test_notify_send_return_val_group(self):
         results = notify.send(self.from_user, recipient=self.to_group, verb="commented", action_object=self.from_user)
         for result in results:
             if result[0] is notify_handler:
@@ -137,7 +136,7 @@ def test_mark_all_as_read_manager(self):
         Notification.objects.filter(recipient=self.to_user).mark_all_as_read()
         self.assertEqual(self.to_user.notifications.unread().count(), 0)
 
-    @override_settings(DJANGO_NOTIFICATIONS_CONFIG={"SOFT_DELETE": True})  # pylint: disable=invalid-name
+    @override_settings(DJANGO_NOTIFICATIONS_CONFIG={"SOFT_DELETE": True})
     def test_mark_all_as_read_manager_with_soft_delete(self):
         # even soft-deleted notifications should be marked as read
         # refer: https://github.com/django-notifications/django-notifications/issues/126
@@ -155,7 +154,7 @@ def test_mark_all_as_unread_manager(self):
         Notification.objects.filter(recipient=self.to_user).mark_all_as_unread()
         self.assertEqual(Notification.objects.unread().count(), self.message_count)
 
-    def test_mark_all_deleted_manager_without_soft_delete(self):  # pylint: disable=invalid-name
+    def test_mark_all_deleted_manager_without_soft_delete(self):
         self.assertRaises(ImproperlyConfigured, Notification.objects.active)
         self.assertRaises(ImproperlyConfigured, Notification.objects.active)
         self.assertRaises(ImproperlyConfigured, Notification.objects.mark_all_as_deleted)
@@ -295,7 +294,7 @@ def test_delete_messages_pages(self):
         self.assertEqual(len(response.context["notifications"]), len(self.to_user.notifications.unread()))
         self.assertEqual(len(response.context["notifications"]), self.message_count - 1)
 
-    @override_settings(DJANGO_NOTIFICATIONS_CONFIG={"SOFT_DELETE": True})  # pylint: disable=invalid-name
+    @override_settings(DJANGO_NOTIFICATIONS_CONFIG={"SOFT_DELETE": True})
     def test_soft_delete_messages_manager(self):
         self.login("to", "pwd")
 
@@ -439,7 +438,7 @@ def test_all_list_api(self):
         self.assertEqual(data["all_list"][0]["verb"], "commented")
         self.assertEqual(data["all_list"][0]["slug"], data["all_list"][0]["id"])
 
-    def test_unread_list_api_mark_as_read(self):  # pylint: disable=invalid-name
+    def test_unread_list_api_mark_as_read(self):
         self.login("to", "pwd")
         num_requested = 3
         response = self.client.get(

notifications/types.py

@@ -2,4 +2,4 @@
 
 from django.contrib.auth.base_user import AbstractBaseUser
 
-AbstractUser = NewType("AbstractUser", AbstractBaseUser)
+AbstractUser = NewType("AbstractUser", AbstractBaseUser)  # type: ignore[valid-newtype]

pyproject.toml

@@ -52,19 +52,20 @@ classifiers = [
 
     "Framework :: Django",
     "Framework :: Django :: 3.2",
-    "Framework :: Django :: 4.0",
-    "Framework :: Django :: 4.1",
+    "Framework :: Django :: 4.2",
+    "Framework :: Django :: 5.0",
     "Programming Language :: Python",
     "Programming Language :: Python :: 3",
     "Programming Language :: Python :: 3.7",
     "Programming Language :: Python :: 3.8",
     "Programming Language :: Python :: 3.9",
     "Programming Language :: Python :: 3.10",
     "Programming Language :: Python :: 3.11",
-    "Topic :: Communications :: Notification",
+    "Programming Language :: Python :: 3.12",
+    "Topic :: Communications",
     "Topic :: Internet",
-    "Topic :: Software Development :: Libraries :: Python Modules",
     "Topic :: Software Development :: Libraries",
+    "Topic :: Software Development :: Libraries :: Python Modules",
     "Topic :: Utilities",
 ]
 packages = [
@@ -118,9 +119,9 @@ ignore-paths = [
 ]
 jobs = 0
 django-settings-module = "notifications.settings"
-fail-on=[
-    "useless-suppression",
-]
+# fail-on=[ # TODO: Investigate why the Pylint results are different in different platforms (macOS and GH actions)
+#     "useless-suppression",
+# ]
 
 [tool.pylint.DESIGN]
 max-locals = 20
@@ -141,9 +142,9 @@ disable = [
 [tool.pylint.FORMAT]
 max-line-length = 120
 
-
 [tool.black]
 line-length = 120
+extend-exclude = "migrations/"
 
 [tool.isort]
 profile = "black"
@@ -180,8 +181,17 @@ testpaths = [
 ]
 DJANGO_SETTINGS_MODULE = "notifications.tests.settings_for_tests"
 django_debug_mode = "keep"
+show_locals=true
 
 [tool.coverage.run]
 relative_files = true
 parallel = true
 branch = true
+
+[tool.coverage.html]
+directory = "coverage_html_report"
+
+[tool.mypy]
+files = "**/*.py"
+exclude = "migration/*"
+ignore_missing_imports=true