Skip to content

Conversation

@tstirrat
Copy link
Contributor

No description provided.

@tstirrat tstirrat requested a review from yuchenshi March 14, 2020 00:38
@yuchenshi
Copy link
Member

LGTM, but do we have any idea why things are behaving like that? Is it the typescript version... or?

@tstirrat
Copy link
Contributor Author

Honestly I have no idea. I've tried to google it.. it just fails on the global version. I'll try npm i -g typescript

@tstirrat
Copy link
Contributor Author

Tried upgrading WS to 3.8.3 and it also broke. so this is some TS version issue :(

Copy link
Member

@yuchenshi yuchenshi left a comment

Choose a reason for hiding this comment

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

Tested manually. It turned out to be just typescript version.

What works: 3.6.x
What doesn't work: 3.8.x and 3.7.x (?)

@tstirrat tstirrat merged commit 476e1d6 into master Mar 16, 2020
@tstirrat tstirrat deleted the ts-use-ws-ts branch March 16, 2020 20:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants