Reduced need for JSONMessage. #19

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.

Sign up for GitHub

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

Jump to bottom

Merged

ioquatix merged 1 commit into main from improved-message-parse

Jun 20, 2024

Member

ioquatix commented Jun 20, 2024 •

edited

Loading

Protocol::WebSocket::JSONMessage is clunky. We can introduce parse directly on message with a coder argument - and generate too.

Types of Changes

New feature.

Contribution

I added tests for my changes.
I tested my changes locally.
I agree to the Developer's Certificate of Origin 1.1.


          Reduced need for JSONMessage.

f222abc

ioquatix merged commit c3e0404 into main

38 of 42 checks passed

ioquatix deleted the improved-message-parse branch

June 20, 2024 03:14

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet