Skip to content
This repository was archived by the owner on Jun 19, 2025. It is now read-only.

Commit 7cdb5b1

Browse files
authored
Merge pull request #3399 from lissyx/fix-rtd-r0.9
Force npm install on RTD and set appropriate PATH value
2 parents 2d04fbe + 617ce14 commit 7cdb5b1

File tree

1 file changed

+4
-0
lines changed

1 file changed

+4
-0
lines changed

doc/conf.py

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,11 @@
3030
# on our CI as in RTD to avoid regressions on RTD that we would not catch on
3131
# TaskCluster
3232
import subprocess
33+
parent = subprocess.check_output("cd ../ && pwd", shell=True).decode().strip()
34+
os.environ["PATH"] = os.path.join(parent, 'node_modules', '.bin') + ':' + os.environ["PATH"]
3335
subprocess.check_call('cd ../ && npm install [email protected] [email protected] @types/[email protected]', shell=True)
36+
subprocess.check_call('env', shell=True)
37+
subprocess.check_call('which typedoc', shell=True)
3438
subprocess.check_call('cd ../ && doxygen doc/doxygen-c.conf', shell=True)
3539
subprocess.check_call('cd ../ && doxygen doc/doxygen-java.conf', shell=True)
3640
subprocess.check_call('cd ../ && doxygen doc/doxygen-dotnet.conf', shell=True)

0 commit comments

Comments
 (0)