-
Notifications
You must be signed in to change notification settings - Fork 180
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
fix(esm): dont depend on __dirname
#1208
Conversation
fucking finally |
@aprendendofelipe Is ESM working now? If so, I could proceed with new, other features and bugfixes |
Everything ok in my tests |
// @ts-expect-error: ignore until esm only | ||
import.meta.url || __dirname | ||
); | ||
const crossRequire = createRequire(resolve('_')); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@aprendendofelipe I've been following the with interest the work on ESM support here. I was reading the code but can't figure out what this does. Specifically, is this underscore (_
) a special directory?
Can you help just so that I earn something today 😉🙏
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It is explained in the thread #1206 (comment)
Resolve #1167