Skip to content

Update Docs: Named Constructors - #134

Merged
zanbaldwin merged 1 commit into
6.xfrom
z/docs-from-constructors
Sep 16, 2026
Merged

zanbaldwin merged 1 commit into
6.xfrom
z/docs-from-constructors

Conversation

@zanbaldwin

Copy link
Copy Markdown
Member

No description provided.

@greptile-apps

greptile-apps Bot commented Sep 16, 2026

Copy link
Copy Markdown

Greptile Summary

  • The updated helper documentation refers to Multi::fromEmbedded(), but that constructor is not available on Multi; the surrounding example uses IPv6::fromEmbedded().

T-Rex validation blocked

  • PHP is not installed in the validation environment, so the focused constructor-path check could not run.

Confidence Score: 4/5

Safe to merge after the non-blocking documentation correction is made.

One documentation issue is established by the changed text and the available constructor definitions. Runtime confirmation could not run because PHP is unavailable.

Files Needing Attention: docs/04-helpers.md

T-Rex T-Rex Logs

What T-Rex did

  • T-Rex attempted to run the embedded validation script from /home/user/repo with the command php trex-artifacts/multi-from-embedded-validation.php, but the shell reported 'php: not found' and the process exited with code 127.
  • The authored test source is trex-artifacts/multi-from-embedded-validation.php, and the run results are captured in the associated output log.
  • It was noted that a public finding should target the actual documented API, IPv6::fromEmbedded(), rather than the non-existent Multi::fromEmbedded().

View all artifacts

T-Rex Ran code and verified through T-Rex

Reviews (1): Last reviewed commit: "docs: 📚 use the strict named constructo..." | Re-trigger Greptile

Comment thread docs/04-helpers.md Outdated
@zanbaldwin
zanbaldwin force-pushed the z/docs-from-constructors branch from e35ce24 to e1a69da Compare September 16, 2026 12:06
factory() is deprecated in 6.1.0, examples show fromProtocol() and its siblings.
@zanbaldwin
zanbaldwin force-pushed the z/docs-from-constructors branch from e1a69da to d675531 Compare September 16, 2026 12:13
@zanbaldwin
zanbaldwin merged commit cf6b1ae into 6.x Sep 16, 2026
24 checks passed
@zanbaldwin
zanbaldwin deleted the z/docs-from-constructors branch September 16, 2026 12:14
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