Skip to content
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use ESM #1750

Closed
thewilkybarkid opened this issue Jun 13, 2024 · 1 comment
Closed

Use ESM #1750

thewilkybarkid opened this issue Jun 13, 2024 · 1 comment

Comments

@thewilkybarkid
Copy link
Member

No description provided.

thewilkybarkid added a commit that referenced this issue Jun 13, 2024
This change replaces ts-node-dev with tsx.

Note that tsx can't watch arbitrary files, so changes to the local .env no longer restart the server.

Refs #1750 wclr/ts-node-dev#312, privatenumber/tsx#181
thewilkybarkid added a commit that referenced this issue Jun 13, 2024
thewilkybarkid added a commit that referenced this issue Jun 13, 2024
thewilkybarkid added a commit that referenced this issue Jun 13, 2024
thewilkybarkid added a commit that referenced this issue Jun 13, 2024
The types for these libraries seem to be wrong, as they don't have named exports.

Refs #1750
thewilkybarkid added a commit that referenced this issue Jun 13, 2024
Using instanceof doesn't work when comparing the same object but loaded through CJS and ESM.

Refs #1750
thewilkybarkid added a commit that referenced this issue Jun 13, 2024
@thewilkybarkid
Copy link
Member Author

thewilkybarkid commented Jun 14, 2024

thewilkybarkid added a commit that referenced this issue Jun 14, 2024
thewilkybarkid added a commit that referenced this issue Aug 27, 2024
thewilkybarkid added a commit that referenced this issue Aug 27, 2024
thewilkybarkid added a commit that referenced this issue Aug 27, 2024
After a few tests, it seems that the plugin has the same behaviour, with the bonus of ordering 'export' declarations.

Refs #1750, trivago/prettier-plugin-sort-imports#270
thewilkybarkid added a commit that referenced this issue Aug 29, 2024
There are a few changes required to get TypeScript/Node.js to work with ESM. Unfortunately, the text-clipper dependency is broken when using ESM and was replaced by a new version on a different registry, which then requires config changes for Jest and TypeScript.

Refs #1750, arendjr/text-clipper#25
thewilkybarkid added a commit that referenced this issue Aug 29, 2024
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

No branches or pull requests

1 participant