Adds PR Template

.github/PULL_REQUEST_TEMPLATE.md

@@ -0,0 +1,36 @@
+# Description
+
+Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
+
+## Issue reference
+Fixes # (issue-number)
+
+## Review
+Before you mark your PR as ready for review, please ensure that you've considered the following:
+- Updated the [CHANGELOG.md](https://github.com/pybop-team/PyBOP/blob/develop/CHANGELOG.md) in reverse chronological order (newest at the top) with a concise description of the changes, including the PR number.
+- Noted any breaking changes, including details on how it might impact existing functionality.
+
+## Type of change
+- [ ] New Feature: A non-breaking change that adds new functionality.
+- [ ] Optimization: A code change that improves performance.
+- [ ] Bug Fix: A non-breaking change that addresses an issue.
+- [ ] Documentation: Updates to documentation or new documentation for new features.
+- [ ] Refactoring: Non-functional changes that improve the codebase.
+- [ ] Style: Non-functional changes related to code style (formatting, naming, etc).
+- [ ] Testing: Additional tests to improve coverage or confirm functionality.
+- [ ] Other: (Insert description of change)
+
+# Key checklist:
+
+- [ ] No style issues: `$ pre-commit run` (or `$ nox -s pre-commit`) (see [CONTRIBUTING.md](https://github.com/pybop-team/PyBOP/blob/develop/CONTRIBUTING.md#installing-and-using-pre-commit) for how to set this up to run automatically when committing locally, in just two lines of code)
+- [ ] All unit tests pass: `$ nox -s tests`
+- [ ] The documentation builds: `$ nox -s docs`
+
+You can run integration tests, unit tests, and doctests together at once, using `$ nox -s quick`.
+
+## Further checks:
+- [ ] Code is well-commented, especially in complex or unclear areas.
+- [ ] Added tests that prove my fix is effective or that my feature works.
+- [ ] Checked that coverage remains or improves, and added tests if necessary to maintain or increase coverage.
+
+Thank you for contributing to our project! Your efforts help us to deliver great software.

CHANGELOG.md

@@ -2,6 +2,7 @@
 
 ## Features
 
+- [#185](https://github.com/pybop-team/PyBOP/pull/185) - Adds a pull request template, additional nox sessions `quick` for standard tests + docs, `pre-commit` for pre-commit, `test` to run all standard tests, `doctest` for docs.
 - [#215](https://github.com/pybop-team/PyBOP/pull/215) - Adds `release_workflow.md` and updates `release_action.yaml`
 - [#204](https://github.com/pybop-team/PyBOP/pull/204) - Splits integration, unit, examples, plots tests, update workflows. Adds pytest `--examples`, `--integration`, `--plots` args. Adds tests for coverage after removal of examples. Adds examples and integrations nox sessions. Adds `pybop.RMSE._evaluateS1()` method
 - [#206](https://github.com/pybop-team/PyBOP/pull/206) - Adds Python 3.12 support with corresponding github actions changes.

conftest.py

@@ -25,6 +25,7 @@ def pytest_addoption(parser):
     parser.addoption(
         "--notebooks", action="store_true", default=False, help="run notebook tests"
     )
+    parser.addoption("--docs", action="store_true", default=False, help="run doc tests")
 
 
 def pytest_terminal_summary(terminalreporter, exitstatus, config):
@@ -41,6 +42,7 @@ def pytest_configure(config):
     config.addinivalue_line("markers", "examples: mark test as an example")
     config.addinivalue_line("markers", "plots: mark test as a plot test")
     config.addinivalue_line("markers", "notebook: mark test as a notebook test")
+    config.addinivalue_line("markers", "docs: mark test as a doc test")
 
 
 def pytest_collection_modifyitems(config, items):
@@ -50,6 +52,7 @@ def pytest_collection_modifyitems(config, items):
         "integration": "integration",
         "plots": "plots",
         "notebooks": "notebooks",
+        "docs": "docs",
     }
     selected_markers = [
         marker for option, marker in options.items() if config.getoption(option)
@@ -63,7 +66,7 @@ def pytest_collection_modifyitems(config, items):
     # If no options were passed, skip all tests
     if not selected_markers:
         skip_all = pytest.mark.skip(
-            reason="Need at least one of --unit, --examples, --integration, or --plots option to run"
+            reason="Need at least one of --unit, --examples, --integration, --docs, or --plots option to run"
         )
         for item in items:
             item.add_marker(skip_all)

docs/conf.py

@@ -4,10 +4,9 @@
 # -- Path setup --------------------------------------------------------------
 import os
 import sys
+from pathlib import Path
 
-root_path = os.path.abspath(os.path.join(os.path.dirname(__file__), ".."))
-sys.path.insert(0, root_path)
-
+sys.path.append(str(Path(".").resolve()))
 from pybop._version import __version__  # noqa: E402
 
 # -- Project information -----------------------------------------------------

noxfile.py

@@ -37,10 +37,15 @@ def coverage(session):
     )
 
 
+@nox.session
+def plots(session):
+    session.install("-e", ".[plot,dev]", silent=False)
+    session.run("pytest", "--plots", "-n", "0")
+
+
 @nox.session
 def integration(session):
     session.install("-e", ".[all,dev]", silent=False)
-    session.install("pytest", "pytest-mock")
     session.run("pytest", "--integration")
 
 
@@ -64,6 +69,49 @@ def notebooks(session):
     )
 
 
+@nox.session(name="tests")
+def run_tests(session):
+    """Run all the tests."""
+    session.install("-e", ".[all,dev]", silent=False)
+    if PYBOP_SCHEDULED:
+        session.run("pip", "install", f"pybamm=={PYBAMM_VERSION}", silent=False)
+    session.run(
+        "pytest", "--unit", "--integration", "--nbmake", "--examples", "-n", "auto"
+    )
+
+
+@nox.session(name="doctest")
+def run_doc_tests(session):
+    """
+    Checks if the documentation can be built, runs any doctests (currently not
+    used).
+    """
+    session.install("-e", ".[plot,docs,dev]", silent=False)
+    session.run("pytest", "--docs", "-n", "0")
+
+
+@nox.session(name="pre-commit")
+def lint(session):
+    """
+    Check all files against the defined pre-commit hooks.
+
+    Credit: PyBaMM Team
+    """
+    session.install("pre-commit", silent=False)
+    session.run("pre-commit", "run", "--all-files")
+
+
+@nox.session(name="quick", reuse_venv=True)
+def run_quick(session):
+    """
+    Run integration tests, unit tests, and doctests sequentially
+
+    Credit: PyBaMM Team
+    """
+    run_tests(session)
+    run_doc_tests(session)
+
+
 @nox.session
 def docs(session):
     """

tests/docs/test_docs.py

@@ -0,0 +1,48 @@
+import sys
+import pytest
+import shutil
+import subprocess
+from pathlib import Path
+
+
+class TestDocs:
+    """A class to test the PyBOP documentation."""
+
+    @pytest.mark.docs
+    def test_docs(self):
+        """
+        Check if the documentation can be built and run any doctests (currently not used).
+
+        Credit: PyBaMM Team
+        """
+        print("Checking if docs can be built.")
+        docs_path = Path("docs")
+        build_path = docs_path / "_build" / "html"
+
+        try:
+            subprocess.run(
+                [
+                    "sphinx-build",
+                    "-j",
+                    "auto",
+                    "-b",
+                    "html",
+                    str(docs_path),
+                    str(build_path),
+                    "--keep-going",
+                ],
+                check=True,
+                capture_output=True,
+            )
+        except subprocess.CalledProcessError as e:
+            print(f"FAILED with exit code {e.returncode}")
+            print(f"stdout: {e.stdout.decode()}")
+            print(f"stderr: {e.stderr.decode()}")
+            sys.exit(e.returncode)
+        finally:
+            # Regardless of whether the doctests pass or fail, attempt to remove the built files.
+            print("Deleting built files.")
+            try:
+                shutil.rmtree(build_path)
+            except Exception as e:
+                print(f"Error deleting built files: {e}")