Skip to content

docs: remove redundant lint counts - #230

Merged
Teakowa merged 1 commit into
mainfrom
docs/issue-229-derived-counts
Aug 25, 2026
Merged

docs: remove redundant lint counts#230
Teakowa merged 1 commit into
mainfrom
docs/issue-229-derived-counts

Conversation

@Teakowa

@Teakowa Teakowa commented Aug 25, 2026

Copy link
Copy Markdown
Contributor

Fixes #229

  • Removes redundant list-preface and registry cardinalities from durable docs and the LintRegistry doc comment.
  • Leaves LintRegistry as the lint inventory owner and keeps named rule IDs, ordering, CLI flags, workflow contracts, and JSON examples explicit.
  • Preserves numeric CLI, protocol, resource, and illustrative payload values.

@Teakowa
Teakowa merged commit 5ca282c into main Aug 25, 2026
14 checks passed
@Teakowa
Teakowa deleted the docs/issue-229-derived-counts branch August 25, 2026 09:27
@github-project-automation github-project-automation Bot moved this from Backlog to Done in Make Workshop Great Again Aug 25, 2026

Teakowa commented Aug 25, 2026

Copy link
Copy Markdown
Contributor Author

Post-merge review — follow-up required

Atomic review complete. CI passed, the registry/list count removals are otherwise scoped correctly, but one Major (acceptance) issue remains.

docs/cli.md still encodes the same list cardinality through “both are repeatable”. Replacing “Two lint-only flags” with “The following lint-only flags ... both are repeatable” removes the numeral but preserves the synchronization obligation: adding a third repeatable lint-only flag makes the sentence stale. Rewrite without cardinality coupling, e.g. “The following lint-only flags configure the registry and are repeatable:”.

The LintRegistry doc comment, docs/README.md, core-workflow wording, stable rule IDs, CLI contracts, and illustrative JSON values are otherwise correct.

Process note: this PR was merged before independent review, with no recorded review submission, while .github#12 is still open. The implementation should have remained reviewable until the acceptance criteria were audited.

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

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

Remove redundant derived cardinalities from durable documentation

1 participant