-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
Bundle is built without errors, but running the bundle results in a variable being undefined #554
Comments
Possibly related to this plugin issue? rollup/rollup-plugin-commonjs#35 (Named) reporting occuring here: https://github.com/SalesforceEng/tough-cookie/blob/a44dcfecc46b72f8e1c5f843052046efb7eb8681/lib/cookie.js#L1325 |
I am new to rollup and faced this issue. Any possible workaround? I am trying to create allinone bundle for nodejs application. |
I believe I am seeing a similar error. I am building a React project with ES6 syntax and when I use a certain file from one of my dependencies(Material-UI), I get the following error in the console when I run the successfully built bundle:
Going to the referenced line, I see something strange:
And my
|
Hi, @gamebox! Can you create repository with minimal reproducing? |
Let me push a commit to this silly little project, and I'll post a link. |
Fixed in rollup/rollup-plugin-commonjs#80 |
Sorry if this report is a bit vague, but I am trying to build a bundle that's gets passed though various plugins. Bundle is built but running it results on following:
The file / module in question is this: https://github.com/SalesforceEng/tough-cookie/blob/master/lib/permuteDomain.js which is used by request in slack-notify.
My
rollup
function looks like this: I am not entirely sure if this is the correct way to approach this. First time using rollup, but my main goal is to have a standalone bundle that can be ran with Node with#!/usr/bin/env node
(so a bundle with my code andnode_modules
included but without Node built-ins).Anything I should be looking into or is something definitely wrong here?
The text was updated successfully, but these errors were encountered: