Skip writing file if redirect is broken (#38)

TheTripleV · web-flow · commit 5f53089aa2c5 · 2020-11-11T18:22:26.000-05:00
diff --git a/sphinxext/rediraffe.py b/sphinxext/rediraffe.py
@@ -242,12 +242,14 @@ def build_redirects(app: Sphinx, exception: Union[Exception, None]) -> None:
                 f'{yellow("(broken)")} {redirect_from} redirects to {redirect_to} but {build_redirect_from} already exists!'
             )
             app.statuscode = 1
+            continue
 
         if not build_redirect_to.exists():
             logger.warning(
                 f'{yellow("(broken)")} {redirect_from} redirects to {redirect_to} but {build_redirect_to} does not exist!'
             )
             app.statuscode = 1
+            continue
 
         build_redirect_from.parent.mkdir(parents=True, exist_ok=True)
         with build_redirect_from.open("w") as f:

Original file line number	Diff line number	Diff line change
`@@ -242,12 +242,14 @@ def build_redirects(app: Sphinx, exception: Union[Exception, None]) -> None:`
`242`	`242`	`f'{yellow("(broken)")} {redirect_from} redirects to {redirect_to} but {build_redirect_from} already exists!'`
`243`	`243`	`)`
`244`	`244`	`app.statuscode = 1`
	`245`	`+ continue`
`245`	`246`
`246`	`247`	`if not build_redirect_to.exists():`
`247`	`248`	`logger.warning(`
`248`	`249`	`f'{yellow("(broken)")} {redirect_from} redirects to {redirect_to} but {build_redirect_to} does not exist!'`
`249`	`250`	`)`
`250`	`251`	`app.statuscode = 1`
	`252`	`+ continue`
`251`	`253`
`252`	`254`	`build_redirect_from.parent.mkdir(parents=True, exist_ok=True)`
`253`	`255`	`with build_redirect_from.open("w") as f:`