foundkey-js: fix build errors #135

Merged
norm merged 1 commit from fix/foundkeyjs-build-errors into main 2022-09-09 23:08:29 +00:00
Owner

The use of "moduleResolution": "Node16" is to make TS use the same
module resolution as Node does when using ESM modules.

reconnecting-websocket's TS definition files were copied over but
wrapped in a declare module block to make TS properly accept
using the default import directly without using .default.

I also decided to look into autobind-decorator's source code and
figured that I could use the boundMethod annotation directly instead
of using the autobind default export.

The use of `"moduleResolution": "Node16"` is to make TS use the same module resolution as Node does when using ESM modules. `reconnecting-websocket`'s TS definition files were copied over but wrapped in a `declare module` block to make TS properly accept using the default import directly without using `.default`. I also decided to look into `autobind-decorator`'s source code and figured that I could use the `boundMethod` annotation directly instead of using the `autobind` default export.
norm force-pushed fix/foundkeyjs-build-errors from a955333919 to dde6f5eb55 2022-09-09 04:08:39 +00:00 Compare
norm merged commit 12bc89b7ce into main 2022-09-09 23:08:29 +00:00
norm deleted branch fix/foundkeyjs-build-errors 2022-09-09 23:08:30 +00:00
Sign in to join this conversation.
No reviewers
No labels
feature
fix
upkeep
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: FoundKeyGang/FoundKey#135
No description provided.