[wip] wit-encoder #1549
Closed
[wip] wit-encoder #1549
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
I started work on a new
wit-encodercrate a few weeks ago. I haven't found time to finish this work up, but @MendyBerger mentioned he might have time to progress some of this work - so I'm putting this up as a draft PR. Part as a reference, part so he can maybe push this forward.Definitely not intended to be merged (yet?). Probably doesn't need to stay open for too long either. I just wanted to make sure to put this up so folks could take a look at it in a place that's easy to reference.
To share the stat of this work: I'd just started implementing the actual backing impls of the various types. To make sure I got this right I was in the progress of porting the parser tests and making sure we're able to generate those programs correctly using
wit-encoder. To handle indentation I was thinking of implementing something similar to thehtml::Rendertrait.Anyway; despite not being done yet I hope it's okay to share this here. Perhaps someone can run with it, or if not I'll find time to pick it up once my current workload lifts. Thanks!