-
Notifications
You must be signed in to change notification settings - Fork 21
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
process.env is empty object #7
Comments
we don't populate that object currently, you actually don't want to do what you suggested by default, or really ever, because there is a lot of stuff in your env and some of it you'd really not want to have bundled up in your client side code. it might make sense to add this as an option though to list the properties you want to include from the env. |
Yeah, that makes sense. There's a lot of junk on the node Something like:
|
Is there a way for the umd to check if it is being required from the commandline, and then proxy the process object? |
if this is running in node...you don't need this library because node provides it all |
@calvinmetcalf if I am using rollup to create a single UMD, don't I need this in case its loaded by the browser? |
this is based on how browserify and webpack do it, but they assume their bundles will only be used in a browser, so you may just want to build 2 bundles, one that includes this for the browser and one that doesn't for the server |
My rollup config plugins:
In my app code when I log
process.env
it has no properties. I would expect to see theMY_PROP
prop which I'm setting in my config, along with all the other props onprocess.env
Am I missing some setup?
The text was updated successfully, but these errors were encountered: