New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: deserialisation and restructuring #7
Conversation
Co-authored-by: Matt Kantor<the.matt.kantor@gmail.com>
So what to do for the 6 PRs I have waiting. Will you merge those into main first and then reconcile with this? |
I'd prefer getting this here in first, this PR already took longer than planned and I would not want to put more weight onto it. |
Ok. Any ETA on the completion of this PR? |
I'm currently running into an error, which should be gone once pmowrer/semantic-release-monorepo#137 is merged. That said, to unblock, I could also delay the multi-package setup and get the deserialization part already merged. Let me sleep on it, I'll make a call tomorrow. |
@brettz9 took a bit longer to sleep on it. I'm pretty busy with other project, so I'll let this PR lie dormant and would be fine with reviewing and eventually merging whatever you have lined up! |
Hi @Gregoor ... Hope all is going well... Any expectations of being able to revisit this PR? It looks like per pmowrer/semantic-release-monorepo#137 (comment) , that there is another option for the semantic release tool? |
Hi @brettz9, thanks I'm good, hope you too! That PR looks interesting, Unfortunately I was not able to get a typed version of (de)serialisation to work, TypeScript is just too much of an unwieldy beast when it comes to recursive types. A better path forward would probably be stripping the type mapping and just shipping the PR without it. I'm also not working on commonbase anymore and don't have use case for zodex myself atm, happy to review a PR that does that though. |
If I do get to a PR, you'd be ok without the monorepo changes--just the addition of the deserialize file in JavaScript form (or possibly TypeScript with some |
I'd be fine without the monorepo changes, though it should be workable without anys, I'd hope. I'm happy to review it, though I don't have merge rights. |
Sorry, it seems it is well above my current TS skills, so short of stripping the types, I don't think I can manage a PR. :-( |
Perhaps this can be closed now, too... |
Added deserialization and restructured the project into a monorepo*.
@brettz9 I've attempt to port all your additions over, I hope non got lost in the merging frenzy.
*that's also the remaining TODO: Will have to adjust release infra to use https://github.com/pmowrer/semantic-release-monorepo to enable per-package publishing.