-
Notifications
You must be signed in to change notification settings - Fork 65
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.js 7.0.0 #1131
Comments
I'm getting an error:
You can see a build log at: https://ci.appveyor.com/project/remojansen/inversifyjs/build/254/job/oq4wfttejd4yrjt8#L307 Looks like this is a known issue npm/npm#14438. |
And how this is related to AppVeyor? |
The following is done by appveyor:
I'm not sure about the right fix but here are some of the suggestions from the issue that I already linked:
Maybe your node installation script is not working properly? |
I still fail to see how this is an AppVeyor only issue. I mean, there shouldn't be any modules in %APPDATA%\npm. As for manually deleting the local node_modules folder, that's something the user has to do. Or the AppVeyor cache should be more clever and check for the node version. /CC @FeodorFitsner |
The build runs in a few versions of node:
The build is working fine in all versions of node except I also found a second related issue with similar suggestions nodejs/node#9377 |
Just clear your node_modules manually. The cache should be more clever and depend on the node version like Travis CI is doing. |
Looks like I fixed by inversify/InversifyJS@6c8b874 but I really think this shouldn't be needed as a manual step. Thanks a lot for helping me out 👍 |
Your so called fix is moot. You are basically disabling the cache. Unless you revert it later that is. |
Have a look at https://github.com/appveyor/website/blob/master/appveyor.yml and especially
|
No description provided.
The text was updated successfully, but these errors were encountered: