- Notifications
You must be signed in to change notification settings - Fork2.4k
Add WebSocketStream#22875
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Add WebSocketStream#22875
Uh oh!
There was an error while loading.Please reload this page.
Conversation
foolip commentedApr 18, 2024
This is an interesting case. Because ofhttps://caniuse.com/websockets andhttps://websockets.spec.whatwg.org/ I would say that "websockets" is the best name for the base feature which goes back ages. But if we follow the style of turning We can always rename it later, but for now I guess I'd go with "web-socket-stream" based on the |
Uh oh!
There was an error while loading.Please reload this page.
Elchi3 commentedApr 22, 2024
Thanks@foolip, I have no strong feelings about the id and accepted your suggestion. Merging this. Do you want me to open a PR on web-features or do we have tooling in place that will signal to web-features that we have a new tag in BCD which will spawn something (automatically) over there? |
foolip commentedApr 22, 2024
@Elchi3 we don't have any tooling for this, but that would make a lot of sense. We could have a test that fails if there are any features in BCD that aren't in web-features, and a list for allowed exceptions which we could treat like a burndown list. |
Discovered by the@openwebdocs BCD collector thanks toopenwebdocs/mdn-bcd-collector#1341.
No spec_url yet, seewhatwg/websockets#48
Adding the "websocket-stream" tag, sounds good Philip?