search

amazon-ion / ion-rust

Rust implementation of Amazon Ion
Apache License 2.0
146 stars 34 forks source link

Reconcile first_commit_candidate Branch with master #14

Open almann opened 4 years ago

almann commented 4 years ago

The first_commit_candidate branch should be reconciled with master. It seems to be further along than master, I suspect we should use it as a first cut for what we want to do in master, but we should remove this branch (or merge it) once we're done with it.

@zslayton is probably best to comment here, but I think it is good to have an issue to track this.

zslayton commented 4 years ago

first_commit_candidate was the original experimental implementation I had put together. It was pretty complete but also very messy. I had started the process of migrating functionality over to master in the form of reasonably-sized, thoroughly-commented PRs, but the work was put on hold. I'd suggest continuing that effort.