Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix: update src/lib.js of @ucanto/interface to have an export statement #332

Closed
wants to merge 1 commit into from

Conversation

gobengo
Copy link
Contributor

@gobengo gobengo commented Nov 17, 2023

could fix #331

TypeError: The specifier “@ucanto/interface” was a bare specifier, but was not remapped to anything. Relative module specifiers must start with “./”, “../” or “/”.

Rationale

  • @ucanto/interface package.json has main set to src/lib.js but that file is empty. and has no exports, which sometimes can confuse js-sniffing tools trying to figure out whether its a module or commonjs or what

@gobengo gobengo self-assigned this Nov 17, 2023
@gobengo gobengo requested review from Gozala and olizilla November 17, 2023 16:54
@gobengo
Copy link
Contributor Author

gobengo commented Nov 17, 2023

happens here through jsdelivr https://jsfiddle.net/

@gobengo
Copy link
Contributor Author

gobengo commented Nov 17, 2023

I don't think this will help it necessarily

@gobengo gobengo closed this Nov 17, 2023
@gobengo gobengo reopened this Nov 17, 2023
@gobengo
Copy link
Contributor Author

gobengo commented Nov 17, 2023

No, this does seem both safe and worth trying so the file contents are obviously a js module.

@gobengo gobengo changed the title fix: update src/lib.js of @ucanto/interface to have an export fix: update src/lib.js of @ucanto/interface to have an export statement Nov 17, 2023
@gobengo
Copy link
Contributor Author

gobengo commented Nov 17, 2023

#331 (comment)

@gobengo gobengo closed this Nov 28, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
1 participant