Skip to content

Update donatj/mddoc dependency version to ^0.13.0 - #14

Merged
donatj merged 3 commits into
masterfrom
mddoc-13
Oct 7, 2026
Merged

donatj merged 3 commits into
masterfrom
mddoc-13

Conversation

@donatj

@donatj donatj commented Oct 6, 2026

Copy link
Copy Markdown
Member

No description provided.

@donatj

donatj commented Oct 6, 2026

Copy link
Copy Markdown
Member Author

@copilot run ./vendor/bin/mddoc and commit the changes to the docs

Co-authored-by: donatj <133747+donatj@users.noreply.github.com>

Copilot AI commented Oct 6, 2026

Copy link
Copy Markdown
Contributor

@copilot run ./vendor/bin/mddoc and commit the changes to the docs

Ran ./vendor/bin/mddoc and committed the generated README changes in 5078421.

@donatj
donatj marked this pull request as ready for review October 6, 2026 22:16
Copilot AI balanced review requested due to automatic review settings October 6, 2026 22:16

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot review overview

🟡 Changes recommended

The generated funtil signature is inaccurate and uses syntax incompatible with supported PHP 7.4.

Review effort: Balanced
Findings: 2 Low severity

Open (2)
What changed in this PR

Updates MDDoc and adds CI verification for generated documentation.

Changes:

  • Upgrades MDDoc to ^0.13.0.
  • Regenerates README signatures and headings.
  • Adds documentation validation to CI.
File Description
composer.json Updates MDDoc dependency.
README.md Applies regenerated documentation formatting.
.github/​workflows/​ci.yml Verifies generated documentation.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread README.md
$stream,
string $until,
int $length = 0,
?string $buf = null,
Comment thread README.md
string $until,
int $length = 0,
?string $buf = null,
): bool
@donatj
donatj merged commit 3a655fa into master Oct 7, 2026
17 checks passed
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.

3 participants