From f5004e6e6257df703808d134b4300b8094b1899a Mon Sep 17 00:00:00 2001
From: Agriya Khetarpal <74401230+agriyakhetarpal@users.noreply.github.com>
Date: Tue, 29 Aug 2023 15:47:24 +0530
Subject: [PATCH] Cleanup PDF build job on Read the Docs

---
 .readthedocs.yaml | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/.readthedocs.yaml b/.readthedocs.yaml
index 4fafb6e7aa..f907ac23d5 100644
--- a/.readthedocs.yaml
+++ b/.readthedocs.yaml
@@ -35,6 +35,7 @@ build:
       - git fetch --unshallow
     # Altered PDF build and upload job, attributed to
     # https://stackoverflow.com/a/76992101/14001839
+    # This also runs on PR builds, but does not upload the PDF
     post_build:
       - mkdir --parents $READTHEDOCS_OUTPUT/pdf/
       # Generate LaTeX files in docs/build/ and doctrees in docs/_build/doctrees, skipping notebooks
@@ -42,11 +43,8 @@ build:
       - cd docs/build/ && cat latexmkrc
       # Map non-zero exit codes to zero
       - cd docs/build/ && latexmk -r latexmkrc -pdf -f -dvi- -ps- -jobname=pybamm -interaction=nonstopmode || true
-      # Check if pybamm.pdf exists
       - test -f docs/build/pybamm.pdf && echo "pybamm.pdf exists. Copying source file to $READTHEDOCS_OUTPUT/pdf/."
-      - ls $READTHEDOCS_OUTPUT/pdf/
       - cp "docs/build/pybamm.pdf" $READTHEDOCS_OUTPUT/pdf/
-      - ls $READTHEDOCS_OUTPUT/pdf/
 
 # Optionally declare the Python requirements required to build your docs
 python: