Skip to content

docs: mention the PHP SDK in the README - #4041

Open
justinmclean wants to merge 1 commit into
apache:masterfrom
justinmclean:docs/readme-php-sdk
Open

docs: mention the PHP SDK in the README#4041
justinmclean wants to merge 1 commit into
apache:masterfrom
justinmclean:docs/readme-php-sdk

Conversation

@justinmclean

Copy link
Copy Markdown
Member

Which issue does this PR address?

N/A

Rationale

The website says Iggy has client libraries for eight languages, and foreign/php is in the repository, but the README lists seven.

What changed?

The README's SDK section listed six languages with published packages, plus C++ named below as work in progress. PHP was absent entirely, so a reader coming from the website counts seven and wonders which one the site means.

Local Execution

  • Passed — one line of the README, no code paths touched.

AI Usage

Claude, in a Cowork session. Reviewed by a human.

The website says Iggy has client libraries for eight languages and the
repository carries foreign/php, but the README named seven: the six with
published packages, plus C++ as work in progress. PHP was missing.

It sits with C++ rather than in the bulleted list, because that list
points at published packages and the PHP extension has none.
@github-actions

github-actions Bot commented Sep 3, 2026

Copy link
Copy Markdown

Thanks for the PR. It is labeled S-waiting-on-review and queued for review.

Slash commands (own line, regular comment) move it around the queue:

  • /ready - back to S-waiting-on-review after addressing feedback
  • /author - flip to S-waiting-on-author while you finish changes
  • /request-review @user-or-team - request a reviewer

See CONTRIBUTING.md for details.

@github-actions github-actions Bot added the S-waiting-on-review PR is waiting on a reviewer label Sep 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-review PR is waiting on a reviewer

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants