Support rebuilds, add pre-commit

.github/workflows/ci.yml

@@ -2,19 +2,18 @@ name: ci
 on: [push, pull_request]
 
 jobs:
+
   check-format:
+
     runs-on: ubuntu-latest
+
     steps:
     - uses: actions/checkout@v2
-    - uses: actions/setup-python@v2
+    - name: Set up Python 3.8
+      uses: actions/setup-python@v2
       with:
         python-version: 3.8
-    - name: Install Black
-      run: |
-        pip install black
-    - name: Run Black
-      run: |
-        black --check --diff .
+    - uses: pre-commit/action@v2.0.0
 
   test:
     name: "Build (${{ matrix.os }} Python ${{ matrix.python-version }})"

.pre-commit-config.yaml

@@ -0,0 +1,22 @@
+# Install pre-commit hooks via
+# pre-commit install
+
+repos:
+
+  # - repo: git://github.com/pre-commit/pre-commit-hooks
+  #   rev: v2.2.3
+  #   hooks:
+  #   - id: check-json
+  #   - id: check-yaml
+  #   - id: end-of-file-fixer
+  #   - id: trailing-whitespace
+
+  # - repo: https://gitlab.com/pycqa/flake8
+  #   rev: 3.7.9
+  #   hooks:
+  #   - id: flake8
+
+  - repo: https://github.com/psf/black
+    rev: 20.8b1
+    hooks:
+    - id: black

sphinxext/rediraffe.py

@@ -117,6 +117,10 @@ def build_redirects(app: Sphinx, exception: Union[Exception, None]) -> None:
     """
     Build amd write redirects
     """
+    try:
+        app.env.redirected
+    except AttributeError:
+        app.env.redirected = {}
 
     if exception != None:
         return
@@ -208,6 +212,19 @@ def build_redirects(app: Sphinx, exception: Union[Exception, None]) -> None:
         build_redirect_from = Path(app.outdir) / redirect_from
         build_redirect_to = Path(app.outdir) / redirect_to
 
+        if (
+            build_redirect_from.exists()
+            and src_redirect_from.as_posix() in app.env.redirected
+        ):
+            # if it is still pointing to the same source, continue
+            if (
+                app.env.redirected[src_redirect_from.as_posix()]
+                == src_redirect_to.as_posix()
+            ):
+                continue
+            # otherwise remove and rewrite
+            build_redirect_from.unlink()
+
         if build_redirect_from.exists():
             logger.warning(
                 f'{yellow("(broken)")} {redirect_from} redirects to {redirect_to} but {build_redirect_from} already exists!'
@@ -240,6 +257,9 @@ def build_redirects(app: Sphinx, exception: Union[Exception, None]) -> None:
             logger.info(
                 f'{green("(good)")} {redirect_from} {green("-->")} {redirect_to}'
             )
+            app.env.redirected[
+                src_redirect_from.as_posix()
+            ] = src_redirect_to.as_posix()
 
 
 class CheckRedirectsDiffBuilder(Builder):

tests/roots/ext/test-simple_rebuild/conf.py

@@ -0,0 +1,8 @@
+extensions = ["sphinxext.rediraffe"]
+
+master_doc = "index"
+exclude_patterns = ["_build"]
+
+html_theme = "basic"
+
+rediraffe_redirects = "redirects.txt"

tests/roots/ext/test-simple_rebuild/index.rst

@@ -0,0 +1 @@
+Index File

tests/roots/ext/test-simple_rebuild/redirects.txt

@@ -0,0 +1 @@
+another.rst index.rst

tests/test_ext.py

@@ -27,6 +27,14 @@ def test_simple(self, app: Sphinx, ensure_redirect):
         assert app.statuscode == 0
         ensure_redirect("another.html", "index.html")
 
+    @pytest.mark.sphinx("html", testroot="simple_rebuild")
+    def test_simple_rebuild(self, app: Sphinx, ensure_redirect):
+        app.build()
+        assert app.statuscode == 0
+        app.build()
+        assert app.statuscode == 0
+        ensure_redirect("another.html", "index.html")
+
     @pytest.mark.sphinx("html", testroot="no_cycle")
     def test_no_cycle(self, app: Sphinx, ensure_redirect):
         app.build()