Skip to content

Emphasize streaming nature of the parser in readme#47

Merged
JanTvrdik merged 1 commit intomainfrom
readme-streaming-emphasis
Feb 24, 2026
Merged

Emphasize streaming nature of the parser in readme#47
JanTvrdik merged 1 commit intomainfrom
readme-streaming-emphasis

Conversation

@JanTvrdik
Copy link
Member

Summary

  • Clarify that this is a streaming parser that reads input in small chunks and yields statements one by one
  • Emphasize it never loads the entire file into memory, making it suitable for large SQL dumps

Clarify that the parser reads input in small chunks and yields
statements one by one, never loading the entire file into memory.
Copilot AI review requested due to automatic review settings February 24, 2026 07:32
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR improves the readme documentation to better emphasize the streaming nature of the SQL parser, clarifying that it processes input in chunks without loading entire files into memory.

Changes:

  • Updated the library description to explicitly state it's a "streaming PHP parser" and explain the memory-efficient processing approach

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@JanTvrdik JanTvrdik merged commit 97d4ae0 into main Feb 24, 2026
16 checks passed
@JanTvrdik JanTvrdik deleted the readme-streaming-emphasis branch February 24, 2026 07:35
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