Austin Ward
Austin Ward
I'll start looking into this issue and add any details that I come across. Would be happy to raise a PR for this one.
@mpalmer The PR for this issue is ready for review when you have the time
@mpalmer no worries, been very busy with work recently. This feedback sounds good, I'm happy to split this apart into two different PRs!
> This looks great :) I think moving the snapshot test suite to Rust makes a lot of sense since it will be easier to maintain and makes it easier...
@bcheidemann thanks for the hand, Ben! I've definitely been distracted with other stuff and haven't had the time to address this feedback. I'm going to work on extracting the testing...
Once #48 is merged, I'll rebase this branch against `main` and update the PR based on the feedback.
Definitely agree, thanks for the quick turn around! I've merged those changes into this branch. Based on feedback from #39, I should be good to remove the `run` binary and...
@mpalmer bumping this in case you missed it
Looks like this was resolved in #21147