Skip to content

Conversation

OskarStark
Copy link
Contributor

Q A
Bug fix? no
New feature? yes
Docs? no
Issues --
License MIT

Extract repeated streaming pattern into reusable print_stream function in examples/bootstrap.php and update all streaming examples to use it.

Extract repeated streaming pattern into reusable print_stream function
in examples/bootstrap.php and update all streaming examples to use it.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
@carsonbot carsonbot changed the title Refactor examples to use print_stream helper function Refactor examples to use print_stream helper function Sep 26, 2025
@OskarStark OskarStark merged commit cb6ed69 into symfony:main Sep 26, 2025
16 checks passed
@chr-hertel
Copy link
Member

chr-hertel commented Sep 26, 2025

naaah, not sure tbh, we're now extracting code from the example that is actually helpful to understand what's going on - i feel like this harms the meaning of the example

same for other print_ functions maybe

@OskarStark OskarStark deleted the refactor-examples-print-stream branch September 26, 2025 11:24
@OskarStark
Copy link
Contributor Author

Ok, so shall I revert?

@chr-hertel
Copy link
Member

we can discuss in #693 - it's not the only one, but let's see how we move forward.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants