french_rev: delete the five local copies, now that the site is published - #827
Conversation
Phase 2 of the french_rev repoint. #826 moved the reads to data-lectures but deliberately kept these files; publish-2026aug07 has now refreshed the site, so nothing reads them any more. Verified before deleting, against the deployed artifact rather than main: the notebook served at intro.quantecon.org/_notebooks/french_rev.ipynb carries four data-lectures references, zero references to lectures/datasets/, and no local-path np.load. The wasm site's french-rev page likewise carries zero references to this repo and five raw.githubusercontent.com reads of data-lectures. An org-wide sweep over every repo's origin/main finds no remaining reader of these five paths; the only matches are data-lectures' own manifests and audit annotations. fig_3.ods stays. It is an orphan rather than part of this set, and unlike wasm's copy it is the last one in the org — data-lectures dropped it as a verified format twin of fig_3.xlsx. Retiring it is Track X's call, with the sweep that covers the other two files left in this directory. Part of QuantEcon/workspace-lectures#23. Completes the set recorded in QuantEcon/data-lectures#49.
There was a problem hiding this comment.
Copilot wasn't able to review any files in this pull request.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
✅ Deploy Preview for taupe-gaufre-c4e660 ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
Correction to this PR's This PR kept So removing it is not a data-retention decision at all — it is an ordinary orphan deletion, and the same is true of the copy sitting in One more thing about this PR worth recording where the claim was made rather than only in the tracker. Its pre-deletion sweep was described as covering every repo's Found by an independent review of QuantEcon/workspace-lectures#23. Nothing here needs undoing in this PR — it is context for the sweep that follows. |
Phase 2 of the
french_revrepoint, and the last step of set 3. #826 moved the reads to data-lectures but deliberately kept these five files in place;publish-2026aug07has now refreshed the site, so they are unreferenced and safe to remove.This is repoint rule 3: repoint, publish, then delete. Deleting in the repoint PR is what 404'd the published
long_run_growthnotebook during set 1, because the site lagsmainuntil apublish*tag is cut.Verified against the deployed artifact, not
mainThat distinction is the whole point of the split, so the check was run on what readers actually fetch:
intro.quantecon.org/_notebooks/french_rev.ipynbdata-lecturesreferenceslectures/datasets/referencesnp.load('datasets/…')quantecon.github.io/lecture-wasm/french-rev/raw.githubusercontent.com/QuantEcon/data-lecturesreadsAn org-wide sweep over every repo's
origin/mainfinds no remaining reader of these five paths. The only matches are data-lectures' own sidecar manifests and its audit annotations, neither of which is a read.fig_3.odsis deliberately keptQuantEcon/lecture-wasm#55 deleted wasm's
fig_3.odsbecause it was a redundant mirror. This repo's copy is different: it is the last one in the org, since data-lectures dropped it during the restructure as a verified format twin offig_3.xlsxwith no consumers. Removing it is a data-retention decision rather than a repoint, so it belongs to the orphan sweep (Track X) alongside the two other files left in this directory, not here.After this
Set 3 is complete.
migration.ymlalready records all five asrepointedwith both consumers, and the strict audit is green.🤖 Generated with Claude Code