Skip to content

CI checks do not cover everything that ships #54

Description

@TMHSDigital

The problem

  • The em-dash check (ci.yml:141) covers only *.md and src/. It skips site/index.html, the page prose in scripts/build_site.py, scripts/og_image.html and the issue templates, and it does not catch — or \u2014. -- is also used as a stand-in in METHODOLOGY.md (lines 68, 159, 194, 359, 397, 401) and in loader output (loader.py:226). There are no real violations today.
  • check_site_links.mjs does not check URLs that JavaScript loads (floor-worker.js, example-run.json).
  • site.yml does not run the link check against the live site after deploy.
  • There is no browser test of the page (see the "[object Object]" bug).

What done looks like

  • The em-dash check covers all shipped text, including the site and templates, and the entities. Decide whether -- as prose punctuation is also banned, and fix the existing ones if so.
  • The link checker covers JS-loaded paths.
  • A post-deploy job runs the link check against the Pages URL.
  • A small Playwright smoke test: load the page, run the calculator once, submit one invalid value.

Activity

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

    ciWorkflows, checks, and release automation

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions