Skip to content

Commit 2d5a4e7

Browse files
mmckyclaude
andcommitted
Address Copilot review: drop dangling nb_repository_url, clear doctrees
- nb_repository_url pointed at lecture-python-programming.fr.notebooks, which does not exist; the theme (launch.py) skips notebook-launch links entirely when the key is unset, so removing it is clean for an HTML-only edition - publish.yml now clears _build/.doctrees after downloading the cache (as ci.yml already did, following lecture-python.myst), removing the one staleness vector in the latest-from-main cache convention Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
1 parent fc22b5b commit 2d5a4e7

2 files changed

Lines changed: 5 additions & 1 deletion

File tree

.github/workflows/publish.yml

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -49,6 +49,9 @@ jobs:
4949
branch: main
5050
name: build-cache
5151
path: _build
52+
- name: Clear stale Sphinx environment
53+
shell: bash -l {0}
54+
run: rm -rf _build/.doctrees
5255
# HTML only for now — no PDF or notebook assets
5356
- name: Build HTML
5457
shell: bash -l {0}

lectures/_config.yml

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,8 @@ sphinx:
4545
header_organisation_url: https://quantecon.org
4646
header_organisation: QuantEcon
4747
repository_url: https://github.com/QuantEcon/lecture-python-programming.fr
48-
nb_repository_url: https://github.com/QuantEcon/lecture-python-programming.fr.notebooks
48+
# nb_repository_url omitted deliberately: no notebooks repo exists for this
49+
# edition yet, and the theme skips notebook-launch links when it is unset
4950
path_to_docs: lectures
5051
twitter: quantecon
5152
twitter_logo_url: https://assets.quantecon.org/img/qe-twitter-logo.png

0 commit comments

Comments
 (0)