-
-
Notifications
You must be signed in to change notification settings - Fork 928
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
Replace @dual-bundle/import-meta-resolve
with import-meta-resolve
#7556
Comments
Upgrading to Node.js 22.0.0 and running |
@boris-petrov I believe you're describing wooorm/import-meta-resolve#27, right? |
@JounQin yes. It's good there is an issue so hopefully it will be fixed at some point. :) |
https://github.com/wooorm/import-meta-resolve/releases/tag/4.1.0 now resolved the deprecation warning. @JounQin Could you merge the upstream change into |
|
In PR #7532, we introduced the
@dual-bundle/import-meta-resolve
package to support both ESM and CJS, which is a fork ofimport-meta-resolve
that is pure ESM.When we switch to pure ESM (maybe in the next major version), we can replace
@dual-bundle/import-meta-resolve
withimport-meta-resolve
.See also the code below:
stylelint/lib/utils/resolveSilent.mjs
Lines 6 to 7 in a211d8b
Note
Node.js has provided the
import.meta.resolve()
API, and we would like to use the API instead ofimport-meta-resolve
, but it is still experimental. In addition, the--experimental-import-meta-resolve
flag is necessary to use the second argumentparent
(at the point of Node.js 21.7.1). So it may be difficult to switchimport.meta.resolve()
in the next major version.The text was updated successfully, but these errors were encountered: