fixup! Show elapsed time for build and publishing

Author: hugovk

.github/workflows/lint.yml

@@ -0,0 +1,22 @@
+name: Lint
+
+on: [push, pull_request, workflow_dispatch]
+
+env:
+  FORCE_COLOR: 1
+  PIP_DISABLE_PIP_VERSION_CHECK: 1
+
+permissions:
+  contents: read
+
+jobs:
+  lint:
+    runs-on: ubuntu-latest
+
+    steps:
+      - uses: actions/checkout@v4
+      - uses: actions/setup-python@v5
+        with:
+          python-version: "3.x"
+          cache: pip
+      - uses: pre-commit/[email protected]

.github/workflows/test.yml

@@ -0,0 +1,39 @@
+name: Test
+
+on: [push, pull_request, workflow_dispatch]
+
+permissions:
+  contents: read
+
+env:
+  FORCE_COLOR: 1
+  PIP_DISABLE_PIP_VERSION_CHECK: 1
+
+jobs:
+  test:
+    runs-on: ${{ matrix.os }}
+    strategy:
+      fail-fast: false
+      matrix:
+        python-version: ["3.9", "3.10", "3.11", "3.12", "3.13"]
+        os: [ubuntu-latest]
+
+    steps:
+      - uses: actions/checkout@v4
+
+      - name: Set up Python ${{ matrix.python-version }}
+        uses: actions/setup-python@v5
+        with:
+          python-version: ${{ matrix.python-version }}
+          allow-prereleases: true
+
+      - name: Install uv
+        uses: hynek/setup-cached-uv@v2
+
+      - name: Install dependencies
+        run: |
+          python -m pip install -U tox
+
+      - name: Tox tests
+        run: |
+          tox -e py

.pre-commit-config.yaml

@@ -0,0 +1,47 @@
+repos:
+  - repo: https://github.com/pre-commit/pre-commit-hooks
+    rev: v4.6.0
+    hooks:
+      - id: check-added-large-files
+      - id: check-case-conflict
+      - id: check-merge-conflict
+      - id: check-toml
+      - id: check-yaml
+      - id: debug-statements
+      - id: end-of-file-fixer
+      - id: forbid-submodules
+      - id: requirements-txt-fixer
+      - id: trailing-whitespace
+
+  - repo: https://github.com/python-jsonschema/check-jsonschema
+    rev: 0.28.6
+    hooks:
+      - id: check-github-workflows
+
+  - repo: https://github.com/rhysd/actionlint
+    rev: v1.7.1
+    hooks:
+      - id: actionlint
+
+  - repo: https://github.com/tox-dev/pyproject-fmt
+    rev: 2.1.3
+    hooks:
+      - id: pyproject-fmt
+
+  - repo: https://github.com/abravalheri/validate-pyproject
+    rev: v0.18
+    hooks:
+      - id: validate-pyproject
+
+  - repo: https://github.com/tox-dev/tox-ini-fmt
+    rev: 1.3.1
+    hooks:
+      - id: tox-ini-fmt
+
+  - repo: meta
+    hooks:
+      - id: check-hooks-apply
+      - id: check-useless-excludes
+
+ci:
+  autoupdate_schedule: quarterly

build_docs.py

@@ -28,7 +28,7 @@
 import json
 import logging
 import logging.handlers
-from functools import total_ordering
+from functools import total_ordering, cache
 from os import readlink
 import platform
 import re
@@ -779,7 +779,9 @@ def is_mac():
         setup_switchers(
             self.versions, self.languages, self.checkout / "Doc" / "build" / "html"
         )
-        logging.info("Build done (%.1f minutes).", (perf_counter() - start_time) / 60)
+        logging.info(
+            "Build done (%.1f minutes).", format_seconds(perf_counter() - start_time)
+        )
 
     def build_venv(self):
         """Build a venv for the specific Python version.
@@ -915,7 +917,8 @@ def copy_build_to_webroot(self):
             for prefix in prefixes:
                 purge(*[prefix + p for p in changed])
         logging.info(
-            "Publishing done (%.1f minutes).", (perf_counter() - start_time) / 60
+            "Publishing done (%.1f minutes).",
+            format_seconds(perf_counter() - start_time),
         )
 
     def should_rebuild(self):
@@ -1151,6 +1154,15 @@ def build_docs(args) -> bool:
     return all_built_successfully
 
 
+@cache
+def format_seconds(seconds: float) -> str:
+    minutes = int(seconds // 60)
+    seconds = round(seconds % 60)
+    if minutes == 0:
+        return f"{seconds}s"
+    return f"{minutes}m{seconds}s"
+
+
 def main():
     """Script entry point."""
     args = parse_args()

pyproject.toml

@@ -0,0 +1,7 @@
+[tool.pytest.ini_options]
+pythonpath = [
+  ".",
+]
+testpaths = [
+  "tests",
+]

tests/test_build_docs.py

@@ -0,0 +1,21 @@
+import pytest
+
+from build_docs import format_seconds
+
+
+@pytest.mark.parametrize(
+    "seconds, expected",
+    [
+        (0.4, "0s"),
+        (0.5, "0s"),
+        (0.6, "1s"),
+        (1.5, "2s"),
+        (30, "30s"),
+        (60, "1m0s"),
+        (185, "3m5s"),
+        (454, "7m34s"),
+        (7456, "124m16s"),
+    ],
+)
+def test_format_seconds(seconds: int, expected: str) -> None:
+    assert format_seconds(seconds) == expected

tox.ini

@@ -0,0 +1,25 @@
+[tox]
+requires =
+    tox>=4.2
+env_list =
+    lint
+    py{313, 312, 311, 310, 39}
+
+[testenv]
+package = wheel
+wheel_build_env = .pkg
+skip_install = true
+deps =
+    -r requirements.txt
+    pytest
+commands =
+    {envpython} -m pytest {posargs}
+
+[testenv:lint]
+skip_install = true
+deps =
+    pre-commit
+pass_env =
+    PRE_COMMIT_COLOR
+commands =
+    pre-commit run --all-files --show-diff-on-failure