colyseus / uWebSockets-express

Express API compatibility layer for uWebSockets.js
https://npmjs.com/package/uwebsockets-express
MIT License
56 stars 13 forks source link

Improved TypeScript support #37

Closed Byloth closed 2 weeks ago

Byloth commented 2 weeks ago

Hi, there. 👋

This PR is related to Issue #36. I still experiencing the error described there, even on v1.3.10.

Useless personal thought: > _Am I -really- the only one using Colyseus with TypeScript?_ 🥲

That's why I opened this PR that fixes this issue.


I -also- updated a couple of versions on the workflows and added a type-checking action when running them...
I think it could always be useful to do one more test before publishing... 😉

Thank you. ❤️

endel commented 2 weeks ago

Thank you so much for the PR @Byloth!! 💙

Unfortunately the tests are currently broken (not your fault!) I think due to closing the server + listening again...