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

node-compat: Allow process.env or import.meta.env #24978

Closed
birkskyum opened this issue Aug 10, 2024 · 1 comment
Closed

node-compat: Allow process.env or import.meta.env #24978

birkskyum opened this issue Aug 10, 2024 · 1 comment

Comments

@birkskyum
Copy link
Contributor

birkskyum commented Aug 10, 2024

I'm trying to run an express app, and have to rewrite all process.env / import.meta.env to Deno.env.get()

Attaching the vars to process / import.meta would allow the app to run as is.

I get a ReferenceError: process is not defined when using the --env-file

There could also be exposure like Deno.env.ENV_VAR as an alternative to Deno.env.get("ENV_VAR") that is easier to do a find-replace for, than meticulously going through an app to add quotes and parenthesis.

@birkskyum
Copy link
Contributor Author

birkskyum commented Sep 15, 2024

Works fine now with the 2.0 rc.

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

No branches or pull requests

2 participants