Skip to content
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(generators): Use cross-platform ES module __dirname #3402

Merged
merged 1 commit into from
Jan 24, 2024
Merged

Conversation

daffl
Copy link
Member

@daffl daffl commented Jan 24, 2024

More ES module shenanigans because they actually don't work well 😞

Copy link

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: 467b3fc
Status: ✅  Deploy successful!
Preview URL: https://dce6a880.feathers.pages.dev
Branch Preview URL: https://file-url-to-path.feathers.pages.dev

View logs

@daffl daffl merged commit 0ac4882 into dove Jan 24, 2024
5 checks passed
@daffl daffl deleted the file-url-to-path branch January 24, 2024 16:44
@fratzinger
Copy link
Member

FYI: This is better since node v20.11.0 / v21.2.0. Once we deprecate lower versions in the future, we can switch to

import.meta.dirname

see https://twitter.com/wesbos/status/1745882792120955268
see nodejs/node#48740

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants