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

feat(npm): add support for resolving "import" conditions to ESM #400

Merged
merged 13 commits into from
Jul 14, 2024

Conversation

joshblack
Copy link
Member

@joshblack joshblack commented Jun 14, 2024

Background information: primer/react#3565

Update how @primer/behaviors is bundled so that files imported using "import" conditions are correctly brought in as ESM files. This change is important for toolchains that resolve using the standardized Node.js behavior for resolving ESM files as, by default, the type of .js files corresponds to the closest type specified in package.json.

Copy link

changeset-bot bot commented Jun 14, 2024

🦋 Changeset detected

Latest commit: 0129c62

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

This PR includes changesets to release 1 package
Name Type
@primer/behaviors Minor

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

@joshblack joshblack changed the title feat(project): emit .mjs files for ESM bundle feat(npm): add support for resolving "import" conditions to ESM Jun 14, 2024
@joshblack joshblack marked this pull request as ready for review June 14, 2024 18:40
@joshblack joshblack requested a review from a team as a code owner June 14, 2024 18:40
broccolinisoup
broccolinisoup previously approved these changes Jul 12, 2024
Copy link
Member

@broccolinisoup broccolinisoup left a comment

Choose a reason for hiding this comment

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

I don't fully grasp the changes but have full confidence in you 💖 Thanks so much!

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.

3 participants