Skip to content

Initial implementation #1

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

Merged
merged 1 commit into from
Jul 10, 2025
Merged

Initial implementation #1

merged 1 commit into from
Jul 10, 2025

Conversation

dsnet
Copy link
Contributor

@dsnet dsnet commented Jul 8, 2025

This adds the initial implementation of "github.com/go-json-experiment/jsonsplit" that dynamically delegates marshal/unmarshal calls to either v1, v2, or both.

@dsnet dsnet requested review from johanbrandhorst and mvdan and removed request for johanbrandhorst and mvdan July 8, 2025 23:28
@dsnet
Copy link
Contributor Author

dsnet commented Jul 8, 2025

\cc @johanbrandhorst hahaha, I can't specify more than 1 reviewer in a private repository.

@dsnet
Copy link
Contributor Author

dsnet commented Jul 8, 2025

If you want to preview the godoc, visit https://carbonite.digitalstatic.ts.net/github.com/go-json-experiment/jsonsplit

Copy link

@johanbrandhorst johanbrandhorst left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is super cool, well done

@dsnet dsnet force-pushed the implementation branch 3 times, most recently from 14c63f5 to 39597a5 Compare July 9, 2025 06:30
This adds the initial implementation of "github.com/go-json-experiment/jsonsplit"
that dynamically delegates marshal/unmarshal calls to either v1, v2, or both.
@dsnet dsnet force-pushed the implementation branch from 39597a5 to d0564de Compare July 9, 2025 06:43
@dsnet dsnet merged commit 4652539 into master Jul 10, 2025
6 checks passed
@dsnet dsnet deleted the implementation branch July 14, 2025 17:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants