esm: fix extensionless import for --type #61
Conversation
At some point this regressed for extensionless imports for esm.
GeoffreyBooth left a comment
GeoffreyBooth left a comment There was a problem hiding this comment.
Not sure if this is related, but don’t forget that extensionless files are supposed to also look for the "type" field too, not only be executable via --type. The example we’d been discussing was TypeScript’s node_modules/typescript/bin/tsc, which is extensionless and could be written in ESM and executed without --type if node_modules/typescript/package.json contains "type": "module".
This comment has been minimized.
This comment has been minimized.
| Yes extensionless support for Update: Actually it seems this is implemented fine already. |
| Updated, I think it has the correct behavior right now |
| failure is unrelated. Will land in 2 hours if there are no objections... necessary for upstreaming |
At some point this regressed for extensionless imports
for esm.