Skip to content

Enables the README stage at severity error - #449

Merged
johnnyt merged 1 commit into
mainfrom
st-gvxj-readme-stage-error
Oct 11, 2026
Merged

johnnyt merged 1 commit into
mainfrom
st-gvxj-readme-stage-error

Conversation

@johnnyt

@johnnyt johnnyt commented Oct 11, 2026

Copy link
Copy Markdown
Member

What

Enables ex_quality's README stage at severity :error, which needs the
ex_quality ~> 0.16 dev dependency.

  • mix.exs: {:ex_quality, "~> 0.15", ...} becomes {:ex_quality, "~> 0.16", ...},
    options unchanged (dev only, runtime: false).
  • mix.lock: only the ex_quality line moves (0.15.1 to 0.16.0). The resolver
    also offered a newer decimal; that line was kept at its current version, so
    no other lock line changes.
  • .quality.exs: readme: [enabled: :auto, severity: :error], with the comment
    block statifier_oban's .quality.exs already carries.
  • docs/quality-gate-changes.md: the ADR-0011 entry for both guarded paths. Its
    Approved-by: line cites the operator's campaign consent of 2026-10-10, which
    pre-approves exactly this change (the 2026-09-23 entry is the precedent for
    that shape).

No file under lib/ changes, and no fragment: changelog.d/README.md lists
quality gate changes among the things that get none.

Provenance

  • .claude/wurk.json and CLAUDE.md: ex_quality 0.16 also ships an opt-in
    Diataxis stage. It stays off here, and its skip line matched neither of the
    manifest's skip lists, so the workflow kit read it as a run-level skip and
    refused the gate. The manifest now classes it as a project-level gap (the
    pattern statifier_oban, opentelemetry_statifier, statifier_datamodel and
    statifier-ui already carry), and CLAUDE.md's skip-classification section
    gives the reason on the same branch, as that section requires. This changes
    no stage and skips nothing: the stage was already off.
  • Using one pattern across the sibling packages was decided by the conductor
    under a standing consent, 2026-10-10.

Gate

Full mix quality green on the rebased head (attested by mix gate.verify):
the README stage reports every part present, the Gate guard stage passes on the
new entry, and mix quality --profile merge (the ADR judge) is clean. Skipped
stages: Diataxis (project-level, above), Gettext and the ADR judge in the bare
run (both declared not applicable).

Review

Review tier: gate. The manifest declares no pre-request review agents; the
diff was reviewed in-turn. No record under docs/adr/ changes.

The ex_quality dev dependency moves to ~> 0.16, the release that ships
the README stage, and .quality.exs enables that stage at severity
:error with the block statifier_oban already carries. The README keeps
its shape as an introduction with a map, and a finding now fails the
gate. The gate change log carries the entry for both guarded paths.

ex_quality 0.16 also adds an opt-in Diataxis stage, which stays off.
The manifest classes its skip as a project-level gap, and CLAUDE.md
gives the reason beside the list. Only the ex_quality lock line moves;
no file under lib/ changes.

Refs: st-gvxj
@johnnyt
johnnyt force-pushed the st-gvxj-readme-stage-error branch from 3b6f701 to 4d7c31b Compare October 11, 2026 15:50
@johnnyt
johnnyt merged commit 506efa3 into main Oct 11, 2026
1 check passed
@johnnyt
johnnyt deleted the st-gvxj-readme-stage-error branch October 11, 2026 15:52
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.

1 participant