-
Notifications
You must be signed in to change notification settings - Fork 286
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
Fix npm warning #776
Fix npm warning #776
Conversation
Apparently, `--omit-dev` should be used over `--production`.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Need to fix a typo
Co-authored-by: Roma <81729714+rtdev-com@users.noreply.github.com>
grep the repo for |
|
@pinheadmz All set! |
CI was failing on node v12 which is unsupported now anyway. Removing that so this docs-only PR will pass integration tests! |
Ugh, @chjj's DNS seed server is down. I'm gonna fix this flakey test by targeting |
Ugh part III: Fine lets just skip the test that requires external DNS lookups, for now. |
Apparently,
--omit=dev
should be used over--production
.