Skip to content

Generator targets the current ecosystem page; review sample drops archived repos - #93

Merged
bdbarnett merged 1 commit into
mainfrom
fix/stale-links
Oct 30, 2026
Merged

bdbarnett merged 1 commit into
mainfrom
fix/stale-links

Conversation

@bdbarnett

Copy link
Copy Markdown
Contributor

Three stale things found during a docs pass.

Generator target. `scripts/generate_sites.py` wrote the ecosystem map to `pydevices/docs/ecosystem.md`. That page now lives at `docs/how-it-works/ecosystem.md`. When a target is missing the generator prints `[SKIP] Ecosystem target missing` and carries on, so the idempotency check produced no diff and passed; the map in that repo had been drifting unchecked, and had been edited by hand to list `omnipython-lvgl`.

Pointing at the real page would have reverted that edit, because `data/repos_db.json` still described the four archived `lvgl-` repos. So the database now has an `omnipython-lvgl` entry (no generated page, not on the portal grid) and a new optional `ecosystem_map` key overrides `portal_grid` for the map only. The four `lvgl-` entries keep their portal cards and pages but leave the map. `profile/README.md` is regenerated; its only change is that the `omnipython-lvgl` blurb matches the flagship page's wording.

Review sample. `cold-eyes-review.yml` listed repos that are archived. Checked every entry with `gh repo view PyDevices/ --json isArchived`: archived were `palettes`, `pdwidgets`, `lvgl-bindings`, `lvgl-python`, `displayif`, `cmods`. All six are removed; the other ten are live.

Dead links in repos_db.json. Checked all 86 distinct URLs (fragments stripped): every non-GitHub-path URL and repo URL returns 200 with `curl -L`, and all 23 `blob/` and `tree/` paths resolve on the repo's main via `gh api repos/PyDevices//contents/` (404 on a known-bad path confirms the check can fail). The two `lvgl-bindings` install-page anchors match headings that exist. No link needed repointing.

Checked

  • Ran the generator in a scratch workspace (this branch plus fresh clones of PyDevices.github.io, pydevices, gallery, mip): `[OK] Ecosystem map -> pydevices/docs/how-it-works/ecosystem.md`, and `git status` is clean in all four clones, so the idempotency check will pass. Before adding the database entry the same run showed the page losing its Tier 3 section.
  • `python3 -m unittest discover -s tests`: 82 tests OK. `ruff check scripts`: passed.

…chived repos from the review sample, and put omnipython-lvgl in the database
@bdbarnett
bdbarnett merged commit d55df7e into main Oct 30, 2026
8 checks passed
@bdbarnett
bdbarnett deleted the fix/stale-links branch October 30, 2026 23:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant