Skip to content

test_pwt.csv: the consumer notes are current again — all four repos repointed - #101

Merged
mmcky merged 1 commit into
mainfrom
test-pwt-notes-current
Aug 18, 2026
Merged

test_pwt.csv: the consumer notes are current again — all four repos repointed#101
mmcky merged 1 commit into
mainfrom
test-pwt-notes-current

Conversation

@mmcky

@mmcky mmcky commented Aug 18, 2026

Copy link
Copy Markdown
Contributor

Removes the six translation consumer notes on test_pwt.csv.yml whose recorded divergence closed today — zh-cn, fr and fa all merged their Track D sync PRs, and a basename sweep over the four live default branches certifies one read per pandas.md and three per polars.md with zero old-URL residue (the fa regression from the stale-snapshot regeneration was restored same day in lecture-python-programming.fa#155; the defect itself is recorded in QuantEcon/action-translation#276).

The consumers header and the Track D comment in migration.yml now record the certified state and the one gate that remains before the source-copy deletion: all four sites' published notebooks must republish from repointed source, since the read is a live fetch at cell execution.

Edit discipline: parsed comparison against main shows consumers as the only differing top-level key (the note-stripping regex was anchored inside the consumers block after an unanchored first attempt matched integrity.upstream.note at the same indent — the exact C1-flip trap, caught by the count assertion). Strict audit exits 0; CATALOG regenerated.

🤖 Generated with Claude Code

…epointed

The six translation consumer notes recorded a divergence (reads the
source repo's raw URL, not this file) that closed today: zh-cn, fr and
fa all merged their sync PRs and a basename sweep over the four live
default branches certifies 1 read per pandas.md and 3 per polars.md
with zero old-URL residue. The consumers header and the Track D
migration comment now record the certified state, the remaining
republish-lag deletion gate, and the two propagation traps hit on the
way (sync opens PRs rather than committing; resync regenerates from
the source PR's merge-time snapshot — QuantEcon/action-translation#276).

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Copilot AI lite review requested due to automatic review settings August 18, 2026 04:02
@mmcky
mmcky merged commit 0599991 into main Aug 18, 2026
5 checks passed
@mmcky
mmcky deleted the test-pwt-notes-current branch August 18, 2026 04:03

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates dataset-tracking metadata to reflect that test_pwt.csv is now fully repointed across the source lecture repo and its three synced translation repos, removing now-stale “divergence” consumer notes and regenerating the catalog accordingly.

Changes:

  • Update migration.yml Track D commentary to record the completed repoints and clarify the remaining deletion gate (republish lag).
  • Remove translation-specific per-consumer note: blocks from lectures/test_pwt.csv.yml now that those consumers read from data-lectures.
  • Regenerate CATALOG.md so the “Used by” column no longer shows translation warning notes for test_pwt.csv.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
migration.yml Updates Track D narrative for test_pwt.csv to reflect completed repoints and remaining deletion gate.
lectures/test_pwt.csv.yml Removes stale translation consumer notes now that all four repos are repointed; updates the explanatory comment block.
CATALOG.md Regenerated to reflect the manifest change (translation consumers no longer annotated with warning notes).

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants