-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
Not works well with Webpack (AMD package name wrong ?) #554
Comments
The AMD module name is |
with
At last of the js file: |
AMD package name is different with npm package name. |
The protobuf.roots feature has just been added. Are you using 6.1.1? Wait, it's not in the minimal runtime yet... need to add it there, too.
Well, the "protobuf" name wasn't available anymore on npm at that time, and that's the only reason it is using something else on npm. I believe this is a general issue that tools should always consider, because this will become more and more standard in the future / as npm grows. |
When using with webpack (generate static code), it always compiled error:
This problem can be resolved by modify the line 4 in generated .js file:
modify to
The text was updated successfully, but these errors were encountered: