countradooku opened a new pull request, #3463: URL: https://github.com/apache/iggy/pull/3463
## Summary Adds `Iggy\Consumer::iterMessages()` for PHP consumers, returning an `Iggy\MessageIterator` that implements PHP's `Iterator` interface and yields `Iggy\ReceiveMessage` instances for `foreach` consumption. The existing callback-based `consumeMessages(callable, int $limit)` path remains unchanged. The README and PHP stubs document the new API, and the PHP SDK tests now cover ordered iterator consumption. Closes #3462 ## Validation - `cargo fmt --all` - `cargo fmt --manifest-path foreign/php/Cargo.toml -- --check` - `php -l foreign/php/iggy-php.stubs.php && php -l foreign/php/tests/IggySdkTest.php` - `docker run php:8.3-cli-bookworm cargo check --manifest-path foreign/php/Cargo.toml` - pre-commit hooks during commit - pre-push hooks during push, including `cargo clippy` ## Notes The full Dockerized PHP PHPUnit suite could not run locally because `foreign/php/Dockerfile.test` installs Debian PHP 8.2 while `foreign/php/composer.json` requires PHP `>=8.3`. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
