Skip to content

Add winnow parser integration#28

Merged
somethingelseentirely merged 2 commits intomainfrom
codex/integrate-bytes-and-view-types-with-winnow
Jul 16, 2025
Merged

Add winnow parser integration#28
somethingelseentirely merged 2 commits intomainfrom
codex/integrate-bytes-and-view-types-with-winnow

Conversation

@somethingelseentirely
Copy link
Contributor

@somethingelseentirely somethingelseentirely commented Jul 16, 2025

Summary

  • add optional winnow feature
  • implement Stream for Bytes with safe iter_offsets
  • add pop_back and pop_front helpers
  • update winnow integration docs
  • tweak tests for new parser API
  • fix as_slice to borrow from &self again
  • use a custom iterator for winnow Stream
  • rename pop to pop_back
  • rewrite winnow::view using safe helpers and add view_elems parser
  • fix tests to exercise Parser trait
  • make view_elems return a Parser for idiomatic winnow usage

Testing

  • ./scripts/preflight.sh

https://chatgpt.com/codex/tasks/task_e_686ed04368e48322952fce4ab85a129b

@somethingelseentirely somethingelseentirely force-pushed the codex/integrate-bytes-and-view-types-with-winnow branch from 687ec8c to b6d75c7 Compare July 16, 2025 15:56
@somethingelseentirely somethingelseentirely merged commit 7ae5a89 into main Jul 16, 2025
1 check passed
@somethingelseentirely somethingelseentirely deleted the codex/integrate-bytes-and-view-types-with-winnow branch July 16, 2025 15:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

codex

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@somethingelseentirely

Comments