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

fix(esbuild): mjs extensions in esm build #46

Merged
merged 1 commit into from
Feb 5, 2024
Merged

Conversation

Anber
Copy link
Owner

@Anber Anber commented Feb 5, 2024

Motivation

ESM should have mjs as a file extension.

@Anber Anber requested a review from layershifter as a code owner February 5, 2024 12:45
Copy link

changeset-bot bot commented Feb 5, 2024

🦋 Changeset detected

Latest commit: f6d367b

The changes in this PR will be included in the next version bump.

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@Anber Anber force-pushed the feature/esbuild-mjs branch from 0fd6f77 to f6d367b Compare February 5, 2024 12:50
@@ -28,15 +28,15 @@
],
"license": "MIT",
"main": "lib/index.js",
"module": "esm/index.js",
"module": "esm/index.mjs",
Copy link
Collaborator

Choose a reason for hiding this comment

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

We probably should also tweak TS config use proper modules and module resolution, but let's handle this later.

@Anber Anber merged commit a31fca3 into main Feb 5, 2024
4 checks passed
@Anber Anber deleted the feature/esbuild-mjs branch February 5, 2024 14:42
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