-
Notifications
You must be signed in to change notification settings - Fork 198
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
moduleResolution as bundle #388
Comments
I am sorry, but the JavaScript ecosystem is ridiculous. The way dependencies and package managers and preprocessers and bundlers interact with each other changes all the time in a way that is not sane and this repository cannot help with that. Maybe we should add a note saying that you are expected to have a PhD in Javascript Sciences before using the package? |
Sorry, I may have broken it here: db53f37 Did not realize that would affect downstream packages. This is the correct setting for modern TypeScript, though. |
It seems like we could change it to |
Just a suggestion, if you think it doesn't apply we can close the issue. |
Nah you're probably not the only person going to experience this issue. I changed it to |
if these changes are confirmed I will no longer need my configuration |
You should still be using NodeNext or Bundler for modern projects. There's no reason to go back if you got it working. But I'm trying to fix it so it'll work both ways. |
I think my commit will fix it. We need another version deployed to test it out. @fiatjaf I ended up ripping out the GitHub actions for JSR because it's not working. I'll fix it later. |
NIPs that are not exported by default depend on the typescript configuration
moduleResolution
set asbundle
.I'm suggesting that it would be a good idea to include information about the need for this configuration in readme.md.
Maybe it's something obvious that I didn't understand, but it took me a while to understand out how to import these modules:
https://njump.me/note1d750tkpzs6lh5260x0z3r7w2n79ek82r9mg0kazmp6nms0ys8k9stvntte
The text was updated successfully, but these errors were encountered: