From da892442373ea3319a2089e99855c7ebfd8ee6ac Mon Sep 17 00:00:00 2001
From: Melissa DeLucchi <delucchi@andrew.cmu.edu>
Date: Mon, 26 Jan 2026 09:48:54 -0500
Subject: [PATCH] Remove sphinx check, improve mypy check/versions.

---
 .../.pre-commit-config.yaml.jinja             | 52 +------------------
 python-project-template/pyproject.toml.jinja  |  2 +
 2 files changed, 4 insertions(+), 50 deletions(-)

diff --git a/python-project-template/.pre-commit-config.yaml.jinja b/python-project-template/.pre-commit-config.yaml.jinja
index 50e6dfe..fbc4d2e 100644
--- a/python-project-template/.pre-commit-config.yaml.jinja
+++ b/python-project-template/.pre-commit-config.yaml.jinja
@@ -122,7 +122,8 @@ repos:
 {%- endif %}
 {%- if mypy_type_checking != 'none' %}
     # Analyze type hints and report errors.
-  - repo: local
+  - repo: https://github.com/pre-commit/mirrors-mypy
+    rev: v1.19.0
     hooks:
       - id: mypy
         name: mypy (python files in src/ and tests/)
@@ -138,55 +139,6 @@ repos:
             "--strict", # Use mypy strict mode to enforce type hints
           {%- endif %}
           ]
-{%- endif %}
-{%- if include_docs and include_notebooks %}
-    # Make sure Sphinx can build the documentation while explicitly omitting 
-    # notebooks from the docs, so users don't have to wait through the execution 
-    # of each notebook or each commit. By default, these will be checked in the 
-    # GitHub workflows.
-  - repo: local
-    hooks:
-      - id: sphinx-build
-        name: Build documentation with Sphinx
-        entry: sphinx-build
-        language: system
-        always_run: true
-        exclude_types: [file, symlink]
-        args:
-          [
-            "-M", # Run sphinx in make mode, so we can use -D flag later
-                  # Note: -M requires next 3 args to be builder, source, output
-            "html", # Specify builder
-            "./docs", # Source directory of documents
-            "./_readthedocs", # Output directory for rendered documents
-            "-T", # Show full trace back on exception
-            "-E", # Don't use saved env; always read all files
-            "-d", # Flag for cached environment and doctrees
-            "./docs/_build/doctrees", # Directory
-            "-D", # Flag to override settings in conf.py
-            "exclude_patterns=notebooks/*,_build", # Exclude notebooks and build dir from pre-commit
-          ]
-{%- elif include_docs %}
-    # Make sure Sphinx can build the documentation without issues.
-  - repo: local
-    hooks:
-      - id: sphinx-build
-        name: Build documentation with Sphinx
-        entry: sphinx-build
-        language: system
-        always_run: true
-        exclude_types: [file, symlink]
-        args:
-          [
-            "-T", # Show full trace back on exception
-            "-E", # Don't use saved env. always read all files.
-            "-b", # Flag to select which builder to use
-            "html", # Use the HTML builder
-            "-d", # Flag for cached environment and doctrees
-            "./docs/_build/doctrees", # directory
-            "./docs", # Source directory of documents
-            "./_readthedocs", # Output directory for rendered documents.
-          ]
 {%- endif %}
     # Run unit tests, verify that they pass. Note that coverage is run against
     # the ./src directory here because that is what will be committed. In the
diff --git a/python-project-template/pyproject.toml.jinja b/python-project-template/pyproject.toml.jinja
index 15dac2c..145b301 100644
--- a/python-project-template/pyproject.toml.jinja
+++ b/python-project-template/pyproject.toml.jinja
@@ -132,6 +132,8 @@ ignore = [
 {% endif -%}
 
 {%- if mypy_type_checking != 'none' %}
+[tool.mypy]
+python_version = "{{py.min(python_versions)}}"
 [tool.setuptools.package-data]
 {{package_name}} = ["py.typed"]
 {%- endif %}