Skip to content

ci: check the README's PHP blocks against the package - #13

Merged
rgfaber merged 1 commit into
mainfrom
juno/doc-snippets
Oct 21, 2026
Merged

rgfaber merged 1 commit into
mainfrom
juno/doc-snippets

Conversation

@rgfaber

@rgfaber rgfaber commented Oct 21, 2026

Copy link
Copy Markdown
Contributor

Purpose (usability): a reader who copies a PHP block out of the README gets code that parses and calls what this package has, and a block that stops doing so fails CI before it ships (macula-io/macula-codebase-checks#8).

  • ci.yml: a docs job calling macula-io/macula-ci-images/.github/workflows/doc-snippets-php.yml pinned by full commit SHA (d25a36c). The workflow has no paths-ignore, so a doc-only change runs it. Packagist publishes by the tag alone, so there is no publish job here to wait on it.
  • The architecture diagram's fence now names what it is (text), which is what the sweep reads a fence for.

Locally the shared check reports 0 failures in 3 blocks (php -l plus PHPStan at level 2 against the package's own autoloader). This repository's CI runs on main and on pull requests, so this draft PR is what runs the new job on a branch.

Refs macula-io/macula-codebase-checks#8

🤖 Generated with Claude Code

…s#8)

A reader who copies a PHP block out of the README gets code that parses and
calls what this package has, and a block that stops doing so fails CI before it
ships. The job calls the shared check pinned by full commit SHA; a doc change
runs it, since this workflow has no paths-ignore. Packagist publishes by the tag
alone, so there is no publish job to wait on it.

The architecture diagram's fence now names what it is (`text`), which is what
the sweep reads a fence for.

Refs macula-codebase-checks#8
@rgfaber
rgfaber merged commit 06f49d0 into main Oct 21, 2026
4 checks passed
@rgfaber
rgfaber deleted the juno/doc-snippets branch October 21, 2026 17: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