-
Notifications
You must be signed in to change notification settings - Fork 787
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
Wrong version of npm module "debug" added to bundle due to "node process" #494
Comments
Could you please update stencil to the latest release? |
I pulled down your repo and updated to the latest stencil. Then changed the debug import to as follows.
The issue appears to be fixed no build or runtime error. |
Hey @jthoms1, thanks for the continuous support (that said upfront ;-) ). Anyway, I had a closer look to the problem again and I created the error scenario in the repo. The repo is now on the latest Stencil version If you might want to have a look that would be awesome! Repo: bitflower/stencil-debug-problem Would love to se this re-opened. Let me know if I can provide anything else! Node: 8.9.4 EDIT: |
Updated the repo to Stencil 0.7.0.... Problem persists :-( |
Same here. Going same path as @bitflower :) |
Hey @jthoms1 I have created a (private) repo for #618 which also showcases this issue. I have added you to the collaborators! Hope it helps! Repo: |
Had the tiny hope of this being fixed after #618 is working now :-) Do you need something else I can provide? |
Hey guys, this keeps popping up everytime I add a module using |
@bitflower I will spend some extra time on this today. Hopefully I can find you a solution. |
It appears that there is a merged PR in |
I am going to close this issue because it should be fixed in the next release of |
Bug repo:
https://github.com/bitflower/stencil-debug-problem
Current behavior:
See repo: importing "debug" doesn't work. I guess to to the fact that the Stencil compiler runs "on the server" (=node/terminal) and the debug module returns the node.js insetad of the browser.js because of that.
Expected behavior:
We can use "debug".
Steps to reproduce:
See repo
The text was updated successfully, but these errors were encountered: