-
Notifications
You must be signed in to change notification settings - Fork 30.1k
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 14 with warning (node:31518) Warning: Accessing non-existent property 'column' of module exports inside circular dependency #32987
Comments
After I updated my deps, vue project looks fixe. |
because this pr only landed in this must be a SEMVER-MAJOR but not in the update docs cc @nodejs/releasers |
@addaleax Can this warning be more friendly, like telling me which module cause the problem. |
@gengjiawen we do have the way to display which module cause the problem use >node --trace-warnings a
(node:19168) Warning: Accessing non-existent property 'foo' of module exports inside circular dependency
at emitCircularRequireWarning (internal/modules/cjs/loader.js:814:11)
at Object.get (internal/modules/cjs/loader.js:825:5)
at Object.<anonymous> (C:\Users\Himself65\Desktop\github\test\cycle\c.js:4:10)
at Module._compile (internal/modules/cjs/loader.js:1185:30)
at Object.Module._extensions..js (internal/modules/cjs/loader.js:1205:10)
at Module.load (internal/modules/cjs/loader.js:1034:32)
at Function.Module._load (internal/modules/cjs/loader.js:923:14)
at Module.require (internal/modules/cjs/loader.js:1074:19)
at require (internal/modules/cjs/helpers.js:72:18)
at Object.<anonymous> (C:\Users\Himself65\Desktop\github\test\cycle\b.js:1:1) another problem is that we not telling this in the v14 docs. |
For anyone run into similar issue, |
Right, I’m surprised that #32797 was not released in 14.0.0. I assumed that it would be, as it was on v14.x-staging before the release. It should be included in the next one, though. /cc @BethGriggs |
It was included (via 1092bb9): https://github.com/nodejs/node/commits/v14.0.0?after=73aa21658dfa6a22c06451d080152b32b1f98dbe+42 |
It's included in release too. Full console log include this (the log is very long, I am not noticed it in first place, thanks @himself65 pointed out). Maybe joining this (node:2658) Warning: Accessing non-existent property 'cat' of module exports inside circular dependency
(Use `node --trace-warnings ...` to show where the warning was created)
(node:2658) Warning: Accessing non-existent property 'cd' of module exports inside circular dependency |
I guess there might be a bug in #32797 then, it is supposed to show the hint after the first warning |
why |
This is a lame glitch out of the blue, without any immediate hints. Has to be addressed somehow. |
@addaleax I think this is reasonable, what do you think ? |
@gengjiawen I don’t have a strong opinion about that. If you think it’s a reasonable change, feel free to open a PR, I guess? :) |
so, what to do? |
@fuchunhui - It depends on where the warnings are coming from. Step 1Run Node with the Step 2Once you identify a package, check if the error has been fixed upstream. For me it was |
As @gengjiawen stated:
I will open a new issue if I can figure out where such an issue belongs - this doesn't seem to belong in the API docs per say as it is not the Node API, but the fact that 30 odd people have given the quoted post a thumbs up suggests to me that I wasn't the only one wondering how I pass a NodeJS ideally
Any idea where such documentation might belong? |
Have similar issue and just do |
This cleared up the majority of the warnings for me. |
You should update lock file:
|
(node:4760) Warning: Accessing non-existent property 'count' of module exports inside circular dependency i encountered this(above warning) problem after i moved my projects to the old computer running on node version-12 to the new computer with node version 14. from the old computer everything worked fine. |
No entendi nada |
The thing that solved for me was deleting the node modules and package.lock file and then running npm install |
I was using node 14 instead of 12, I moved to version 12 and it resolved the issue for me. For moving from 14 to I used nvm(node version manager). |
The bug wasnt a bug its your imports that rounds around the module exports , should double check and prevent circulating calls of module export functions inside. |
Even i face the same issue.even after npm update i could see the below warnings. Building Angular Package It is not recommended to publish Ivy libraries to NPM repositories. |
This is all I needed. |
Hi friend, I had the same problem and this is how I solved this problem. Step 1 I updated the npm. Step 2 I updated the node js Step 3 When running the Step 4 Then I run this code Final step Finally, I run the program with the command |
|
Summary: Changelog: [Internal] - shelljs < 0.8.4 triggers warnings for node 14 and above. See nodejs/node#32987 (comment) Reviewed By: ShikaSD Differential Revision: D33121070 fbshipit-source-id: 04d5e2c4295186fbad542bbc6085e491fc0d9d21
Summary: Changelog: [Internal] - shelljs < 0.8.4 triggers warnings for node 14 and above. See nodejs/node#32987 (comment) Reviewed By: ShikaSD Differential Revision: D33121070 fbshipit-source-id: 04d5e2c4295186fbad542bbc6085e491fc0d9d21
Thanks bro !,very useful :) |
I'm doing my best to use the solutions listed here, but I'm having issues with the module being found when running |
Happens in lots of library I use.
What steps will reproduce the bug?
Repo:
https://github.com/gengjiawen/sample
CI:
https://github.com/gengjiawen/sample/runs/607139469?check_suite_focus=true#step:4:113
How often does it reproduce? Is there a required condition?
Always.
What is the expected behavior?
no warning
What do you see instead?
Additional information
cc @nodejs/modules-active-members
The text was updated successfully, but these errors were encountered: