[Administration] Upgrade npm packages and use new GraphQL code generator #558
Conversation
| "generate-protobuf:win32": "mkdir src\\generated | pbjs -t static-module -w es6 -o src\\generated\\compiled.js ..\\specs\\card_activate_model.proto --force-long | pbts -o src\\generated\\compiled.d.ts src\\generated\\compiled.js", | ||
| "postinstall": "npm run generate-protobuf" | ||
| "generate-protobuf:linux": "mkdir -p src/generated && pbjs -t static-module -w es6 -o src/generated/protobuf.js ../specs/card_activate_model.proto --force-long && pbts -o src/generated/protobuf.d.ts src/generated/protobuf.js", | ||
| "generate-protobuf:win32": "mkdir src\\generated | pbjs -t static-module -w es6 -o src\\generated\\protobuf.js ..\\specs\\card_activate_model.proto --force-long | pbts -o src\\generated\\protobuf.d.ts src\\generated\\protobuf.js", |
There was a problem hiding this comment.
Should we remove that? Only difference is backslashes. WSL should handle that.
There was a problem hiding this comment.
I'd just keep it for now until it breaks something for us.
|
Merge if you feel comfortable. |
|
The only thing I am uncomfortable with is the annoying issue zxing-js/library#499 which makes the zxing library shout a huge number of warnings (involving source maps) when building. |
I left a comment in that issue to give it a bump. I would say we just merge with warnings and ignore them. Having warnings is better than not having source maps :D |
|
Seems good to me! |
No description provided.