search

mtreinish / subunit-rust

A subunit v2 protocol implementation in Rust
Apache License 2.0
2 stars 2 forks source link

Streaming parser #15

Closed jelmer closed 3 weeks ago

jelmer commented 1 month ago

It would be great if the parser would iterate oven events rather than read all of them at once.

mtreinish commented 1 month ago

Is this a duplicate of https://github.com/mtreinish/subunit-rust/issues/5 ?

jelmer commented 1 month ago

It's a slightly different problem, but indeed probably fixed by the same change.

rbtcollins commented 1 month ago

I have a patch mostly done, doing some code golf - should get back to it next weekend.

rbtcollins commented 1 month ago

https://github.com/rbtcollins/subunit-rust/commit/8288090ac801dac4519a0d4207a35108bb493ca5 - theres a few patches to take, I'll make the PRs as rapidly as @mtreinish chooses to review them :)