-
Notifications
You must be signed in to change notification settings - Fork 87
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
require('google-gax') returns errors #249
Comments
@JustinBeckwith PTAL! |
Just adding notes. I have v8.11.1 and it works fine. |
I tried that version as well as v10.4.1, as well as on another computer (OS X, previously tested on Windows). Same error. Note that this doesn't happen if you just |
I just retried and am seeing issues in, v8.11.3, v10.4.1
|
I have a suspicion this line has something to do with it: Digging deeper. |
So my investigation led me here: I'm not super familiar with that codebase, so I'm not sure how that's supposed to work. If I add a leading |
I've been looking into this, I should have a PR later today. |
Fixed in googleapis/nodejs-proto-files#46! |
I installed only
google-gax
in a fresh directory (Node v10/npm v6.1.0), and tried to require the module from the Node console:The text was updated successfully, but these errors were encountered: