feat: Add healthcheck views + urls, typing, ruff linting, src layout (#13)

Author: khvn26

.github/workflows/linting-and-formatting.yml

@@ -1,12 +1,12 @@
-name: Linting & formatting
+name: Python Checks and Unit Tests 
 
 on:
   pull_request:
 
 jobs:
   test:
     runs-on: ubuntu-latest
-    name: Linting and Formatting
+    name: Python Checks and Unit Tests
 
     steps:
       - name: Cloning repo
@@ -20,10 +20,13 @@ jobs:
           python-version: 3.12
 
       - name: Install Dependencies
-        run: poetry install
+        run: poetry install --with dev
 
-      - name: Run Linters
-        run: | 
-          poetry run black --check .
-          poetry run isort --check-only --diff .
-          poetry run flake8
+      - name: Check for missing migrations
+        run: poetry run python manage.py makemigrations --no-input --dry-run --check
+
+      - name: Check for new typing errors
+        run: poetry run mypy .
+
+      - name: Run Tests
+        run: poetry run pytest

.pre-commit-config.yaml

@@ -1,21 +1,28 @@
 repos:
-    - repo: https://github.com/PyCQA/isort
-      rev: 5.13.2
-      hooks:
-          - id: isort
-            name: isort (python)
-    - repo: https://github.com/psf/black
-      rev: 24.4.2
-      hooks:
-          - id: black
-            language_version: python3
-    - repo: https://github.com/pycqa/flake8
-      rev: 7.1.0
-      hooks:
-          - id: flake8
-    - repo: https://github.com/python-poetry/poetry
-      rev: 1.7.1
-      hooks:
-        - id: poetry-check
-        - id: poetry-lock
-          args: ['--check']
+  - repo: https://github.com/astral-sh/ruff-pre-commit
+    # Ruff version.
+    rev: v0.11.0
+    hooks:
+      # Run the linter.
+      - id: ruff
+        args: [--fix]
+      # Run the formatter.
+      - id: ruff-format
+  - repo: https://github.com/python-poetry/poetry
+    rev: 2.1.1
+    hooks:
+      - id: poetry-check
+      - id: poetry-lock
+  - repo: local
+    hooks:
+      - id: python-typecheck
+        name: python-typecheck
+        language: system
+        entry: poetry run mypy .
+        require_serial: true
+        pass_filenames: false
+        types: [python]
+
+ci:
+  skip: [python-typecheck]
+  autoupdate_commit_msg: "ci: pre-commit autoupdate"

manage.py

@@ -0,0 +1,17 @@
+#!/usr/bin/env python
+import os
+import sys
+
+if __name__ == "__main__":
+    os.environ.setdefault("DJANGO_SETTINGS_MODULE", "settings.dev")
+
+    try:
+        from django.core.management import execute_from_command_line
+    except ImportError:
+        raise ImportError(
+            "Couldn't import Django. Are you sure it's installed and "
+            "available on your PYTHONPATH environment variable? Did you "
+            "forget to activate a virtual environment?"
+        )
+
+    execute_from_command_line(sys.argv)

poetry.lock

@@ -1,10 +1,10 @@
 [tool.poetry]
 name = "flagsmith_common"
-version = "1.4.2"
+version = "1.5.0"
 description = "A repository for including code that is required in multiple flagsmith repositories"
 authors = ["Matthew Elwell <[email protected]>"]
 readme = "README.md"
-packages = [{ include = "common"}]
+packages = [{ include = "common", from = "src" }]
 
 [tool.poetry.dependencies]
 python = "^3.10"
@@ -13,49 +13,66 @@ djangorestframework = "*"
 drf-writable-nested = "*"
 flagsmith-flag-engine = "*"
 djangorestframework-recursive = "*"
+django-health-check = "^3.18.3"
 
 [tool.poetry.group.dev.dependencies]
 pre-commit = "*"
-flake8 = "*"
-black = "*"
-isort = "*"
+ruff = "*"
+pytest = "^8.3.4"
+pyfakefs = "^5.7.4"
+pytest-django = "^4.10.0"
+mypy = "^1.15.0"
+django-stubs = "^5.1.3"
+djangorestframework-stubs = "^3.15.3"
 
 [build-system]
 requires = ["poetry-core"]
 build-backend = "poetry.core.masonry.api"
 
-[tool.black]
+[tool.pytest.ini_options]
+addopts = ['--ds=settings.dev', '-vvvv', '-p', 'no:warnings']
+console_output_style = 'count'
+
+[tool.ruff]
 line-length = 88
-target-version = ['py311', 'py312']
-include = '\.pyi?$'
-exclude = '''
-/(
-    \.eggs
-  | \.git
-  | \.hg
-  | \.mypy_cache
-  | \.tox
-  | \.venv
-  | _build
-  | buck-out
-  | build
-  | dist
-  | migrations
-)/
-'''
-
-[tool.isort]
-use_parentheses=true
-multi_line_output=3
-include_trailing_comma=true
-line_length=79
-known_first_party=['flagsmith','api','app','core','features','environments']
-known_third_party=[
-  'django',
-  'rest_framework',
-  'saml2',
-  'drf_yasg2',
-  'pytest',
-  'rest_framework_recursive',
-]
-skip = ['migrations', '.venv']
+indent-width = 4
+target-version = "py311"
+extend-exclude = ["migrations"]
+
+[tool.ruff.format]
+# Like Black, use double quotes for strings.
+quote-style = "double"
+
+# Like Black, indent with spaces, rather than tabs.
+indent-style = "space"
+
+# Like Black, respect magic trailing commas.
+skip-magic-trailing-comma = false
+
+# Like Black, automatically detect the appropriate line ending.
+line-ending = "auto"
+
+# Enable auto-formatting of code examples in docstrings. Markdown,
+# reStructuredText code/literal blocks and doctests are all supported.
+docstring-code-format = true
+
+# Set the line length limit used when formatting code snippets in
+# docstrings.
+docstring-code-line-length = "dynamic"
+
+[tool.ruff.lint]
+# Establish parity with flake8 + isort
+select = ["C901", "E4", "E7", "E9", "F", "I", "W"]
+ignore = []
+fixable = ["ALL"]
+unfixable = []
+
+[tool.mypy]
+plugins = ["mypy_django_plugin.main"]
+strict = true
+
+[tool.django-stubs]
+django_settings_module = "settings.dev"
+
+[tool.drf-stubs]
+enabled = true

pyproject.toml

@@ -0,0 +1,16 @@
+# Settings expected by `mypy_django_plugin`
+SENDGRID_API_KEY: str
+AWS_SES_REGION_ENDPOINT: str
+SEGMENT_RULES_CONDITIONS_LIMIT: int
+
+# Settings required for tests
+INSTALLED_APPS = [
+    "django.contrib.auth",
+    "django.contrib.contenttypes",
+]
+DATABASES = {
+    "default": {
+        "ENGINE": "django.db.backends.sqlite3",
+        "NAME": "common",
+    }
+}

common/__init__.py renamed to settings/__init__.py

@@ -0,0 +1,13 @@
+from django.urls import include, re_path
+
+from common.app import views
+
+urlpatterns = [
+    re_path(r"^version/?", views.version_info),
+    re_path(r"^health/liveness/?", views.version_info),
+    re_path(r"^health/readiness/?", include("health_check.urls", namespace="health")),
+    re_path(r"^health", include("health_check.urls", namespace="health-deprecated")),
+    # Aptible health checks must be on /healthcheck and cannot redirect
+    # see https://www.aptible.com/docs/core-concepts/apps/connecting-to-apps/app-endpoints/https-endpoints/health-checks
+    re_path(r"^healthcheck", include("health_check.urls", namespace="health-aptible")),
+]