Skip to content

Review the readmes #931

Closed
Closed

Description

@alistairjcbrown

Review the readmes in both the scripts and clusterflick repos.
Are any updates needed, any data out of date, etc?

Activity

  1. changed the title [-]Review the readme[/-] [+]Review the readmes[/+] on Nov 11, 2026
  2. alistairjcbrown commented on Nov 11, 2026

    @alistairjcbrown
    MemberAuthor
  3. alistairjcbrown commented on Nov 11, 2026

    @alistairjcbrown
    MemberAuthor

    Done

    • scripts: draft PR Bring README up to date with the pipeline #932 updates the README. It adds the missing env vars (OpenAI, which is needed by default, plus TypeSafe and Cinebur), documents transform-prepare, registry, departed, health and llm-usage-report, adds clear:llm-usage-data, and links docs/.
    • clusterflick.com: I can only read this repo from here, so I couldn't open a PR. These are the stale bits I found:
      1. Tech Stack and Features say react-virtualized. The site now uses react-virtuoso (package.json, CLAUDE.md).
      2. "Data is refreshed automatically every morning" is out of date. The org profile says listings are pulled several times a day.
      3. Data Access points to the About page for downloads. Downloads now live at /datasets, and the licence is at /data-licence (CC BY 4.0).
      4. Data Pipeline step 5 (Matching) has no repo link, and Related Repositories leaves out data-matched. That's the repo the site reads ratings from.
      5. Other Commands leaves out npm test (vitest unit tests), which Tech Stack mentions, and the fetch-* scripts that CLAUDE.md lists.
      6. Project Structure and Features are missing many newer pages: films, film-clubs, planner, watchlist/personalise, lists, collections, genres, formats, cinema-groups, near-me, datasets, changelog.
    • .github profile README: I read it too and found nothing obviously wrong.

    Uncertain

    • I took "clusterflick repo" to mean clusterflick.com.
    • In the scripts README I invoke registry and departed with node index.js because neither has an npm script. Say if you'd prefer I add npm scripts for them.
    • I haven't checked the "450+ venues" figure. There are 512 venue directories, but some may be closed or inactive.

    Decision needed

    • Should I apply the clusterflick.com fixes? That needs push access for this session, or for you to apply them.

    Generated by Claude Code

  4. alistairjcbrown commented on Nov 11, 2026

    @alistairjcbrown
    MemberAuthor

    clusterflick.com fixes are in draft PR clusterflick/clusterflick.com#231 (README only). It covers all six points from my last comment. Two PRs to review: #932 here and that one.


    Generated by Claude Code

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

Metadata

Metadata

Assignees

No one assigned

    Labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions