-
Notifications
You must be signed in to change notification settings - Fork 15
Conversation
Signed-off-by: rshiv <reeshav96@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good start. Have left some comments and a question re upgrading.
Co-authored-by: Hanno Cornelius <68783915+jm-clius@users.noreply.github.com>
Signed-off-by: rshiv <reeshav96@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks good to me! :) but I'd wait for Hanno and Oskar to approve.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some comments on clarity. In general one idea per sentence may be easier to understand (e.g. having a single sentence explain that we can support both secure and unsecure websockets and the context in which they may be used may become ambiguous/difficult to parse).
Signed-off-by: rshiv <reeshav96@gmail.com>
Signed-off-by: rshiv <reeshav96@gmail.com>
Co-authored-by: oskarth <ot@oskarthoren.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Signed-off-by: rshiv reeshav96@gmail.com