Skip to content
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

ws: make acceptable() more robust #404

Merged
merged 1 commit into from May 14, 2019

Conversation

3 participants
@ry
Copy link
Contributor

commented May 14, 2019

cc @keroxp

@ry ry requested a review from piscisaureus May 14, 2019

@keroxp
Copy link
Contributor

left a comment

LGTM with nits.

Show resolved Hide resolved ws/mod.ts
@piscisaureus
Copy link
Contributor

left a comment

LGTM

@ry ry force-pushed the ry:ws_acceptable branch from a074287 to bf3afcd May 14, 2019

@ry ry merged commit 9e8e6f9 into denoland:master May 14, 2019

5 checks passed

denoland.deno_std Build #20190514.15 succeeded
Details
denoland.deno_std (Linux) Linux succeeded
Details
denoland.deno_std (Mac) Mac succeeded
Details
denoland.deno_std (Windows) Windows succeeded
Details
license/cla Contributor License Agreement is signed.
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.