Skip to content

fix(register): handle require(<json>)#374

Merged
pi0 merged 2 commits intounjs:mainfrom
kricsleo:fix/node-register-require-json
Jul 23, 2025
Merged

fix(register): handle require(<json>)#374
pi0 merged 2 commits intounjs:mainfrom
kricsleo:fix/node-register-require-json

Conversation

@kricsleo
Copy link
Copy Markdown
Member

resolves #355

Not sure if this is the right fix, but all tests passed.
Looking forward to your feedback! ❤️

@kricsleo kricsleo marked this pull request as draft April 16, 2025 15:24
@kricsleo kricsleo marked this pull request as ready for review April 16, 2025 16:19
Copy link
Copy Markdown
Member

@pi0 pi0 left a comment

Choose a reason for hiding this comment

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

Thanks 🚀

@pi0 pi0 changed the title fix(register): handle require JSON fix(register): handle require(.json) Jul 23, 2025
@pi0 pi0 changed the title fix(register): handle require(.json) fix(register): handle require(<json>) Jul 23, 2025
@pi0 pi0 merged commit 54461d6 into unjs:main Jul 23, 2025
6 of 7 checks passed
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.

ERR_REQUIRE_ESM with jiti/register

2 participants