Skip to content

Commit 894a773

Browse files
committed
[no ci] docs: do not build on PRs
Building the docs on PRs requires additional logic, which is not in place yet.
1 parent 208d0f2 commit 894a773

File tree

2 files changed

+5
-5
lines changed

2 files changed

+5
-5
lines changed

.github/workflows/multiversion-docs.yaml

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -3,9 +3,6 @@
33
name: multiversion-docs
44

55
on:
6-
pull_request:
7-
branches:
8-
- dev
96
workflow_dispatch:
107
push:
118
branches:

docsrc/poly.py

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
import asyncio
22
from asyncio.subprocess import PIPE
3-
from logging import getLogger
3+
import logging
44
from subprocess import CalledProcessError
55
import os
66
from pathlib import Path
@@ -22,7 +22,9 @@
2222
cast,
2323
)
2424

25-
logger = getLogger(__name__)
25+
logging.basicConfig()
26+
logger = logging.getLogger(__name__)
27+
logger.setLevel(logging.DEBUG)
2628

2729
#: Whether to build the docs in parallel
2830
PARALLEL_BUILDS = os.environ.get("BF_DOCS_SEQUENTIAL_BUILDS", "0") != "1"
@@ -209,6 +211,7 @@ async def __aenter__(self):
209211

210212
self.logger.debug("Installation output:\n %s", out)
211213
if process.returncode != 0:
214+
logger.error(err)
212215
raise BuildError from CalledProcessError(cast(int, process.returncode), " ".join(cmd), out, err)
213216
return self
214217

0 commit comments

Comments
 (0)