Skip to content

Support reading raw non json strings#62

Open
wlezzar wants to merge 1 commit intoadevinta:masterfrom
wlezzar:rename-jsonifier
Open

Support reading raw non json strings#62
wlezzar wants to merge 1 commit intoadevinta:masterfrom
wlezzar:rename-jsonifier

Conversation

@wlezzar
Copy link
Contributor

@wlezzar wlezzar commented Apr 10, 2025

Fixes: #61

@wlezzar
Copy link
Contributor Author

wlezzar commented Apr 10, 2025

@LouisVN

Copy link
Contributor

@LouisVN LouisVN left a comment

Choose a reason for hiding this comment

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

Hey @wlezzar, thanks a lot for this!

Looks great, perhaps we could add a small test? I suppose we could tweak this test helper that only supports JSON atm?
If you'd like, I can try to write one 😄

@wlezzar
Copy link
Contributor Author

wlezzar commented Apr 22, 2025

Makes perfect sense! I can add the tests 👍

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.

Consume string messages

2 participants