-
-
Notifications
You must be signed in to change notification settings - Fork 18
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
Installation from npm still fails #94
Comments
@feritarou I did some research on this and I think the problem is that the CI system does not now the github.com host. Locally, you usually get a prompt where you can accept the host and then it is added to the The question now is, if it makes more sense to adjust the CI process, or if you should avoid fork dependencies in References: |
I got almost the same error. Maybe it's even the same, but just expressed differently because of a different npm version.
This happens within a GitHub action. A few workarounds are provided in actions/setup-node#214 An easy one being to replace |
@davidsandoz Thanks for the link! I could reproduce the issue and commented on it in the linked issue. I think the only reasonable fix is to replace the forks with scoped packages. It's far from ideal because you cannot make out which scoped package comes from a fork and which is really a scoped production package, but yeah that's how it is. Let me know if that makes sense, then I can do the replacements, no big deal. |
@dword-design I agree with you regarding the reasonable fix. The workaround I applied works, but it doesn't seem like a proper solution to have to update this manually in the package-lock file. |
@davidsandoz Should work after |
Thanks, @dword-design! Works well now 👍 |
Continuing the discussion in the closed Issue #15, nuxt-mail fails to install its dependencies for me in production CI, but in my case it is not related to a missing
git
binary.This is what I get:
I am using Nuxt.js 2.15.8 and deploying to a self-hosted Dokku environment.
The text was updated successfully, but these errors were encountered: