Skip to content
This repository has been archived by the owner on Jul 31, 2018. It is now read-only.

ep: ws in core #6

Closed
wants to merge 2 commits into from
Closed

ep: ws in core #6

wants to merge 2 commits into from

Conversation

eljefedelrodeodeljefe
Copy link

EP as on nodejs/node#4842


> `websockets` are the primary reason for companies, which have **not** decided
to make a commitment towards `node` to use `node` anyways. For a wider adoption
of the language this is a critical point.

Choose a reason for hiding this comment

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

How much basis is there for this? My insight indicates to me: little to none..

Choose a reason for hiding this comment

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

...me as an engineering manager in a PHP-shop.

Choose a reason for hiding this comment

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

Could change wording, but I think this fact still hold up.

Copy link
Member

Choose a reason for hiding this comment

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

I highly doubt that this is a valid point. Without performing some kind of relevant research, at least.

Copy link

Choose a reason for hiding this comment

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

If companies are already adopting node.js because of how good the ws package is, then that means the ws maintainers are doing a great job and there's nothing that node core needs to do.

@trevnorris
Copy link
Contributor

@eljefedelrodeodeljefe Now that v8_inspector has landed, with basic ws support, may want to revise this proposal.

@styfle
Copy link
Member

styfle commented Mar 19, 2018

The most recent discussion is happening in nodejs/node#19308

@Trott
Copy link
Member

Trott commented Jun 12, 2018

Superseded by nodejs/node#19308

@Trott Trott closed this Jun 12, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants