Skip to content

Commit

Permalink
grpc -> grpcjs + generated ts types from proto files
Browse files Browse the repository at this point in the history
notes:
 * grpcjs doesnt like ByteBuffer, it has been replaced with the nodejs
   native Buffer
 * grpcjs doesnt like calls without an argument, add {} instead
   see src/grpc/subscribe.ts:24
 * grpcjs converts uint64 in a map key (map<uint64, ...>) to some hash,
   this is fixed with protobuf_long.patch which is automatically applied
   after running `npm install`
   this pr would fix the bug upstream protobufjs/protobuf.js#1669
  • Loading branch information
antonilol committed Sep 16, 2022
1 parent eb58ea3 commit 79d85b6
Show file tree
Hide file tree
Showing 514 changed files with 18,089 additions and 1,278 deletions.
10 changes: 8 additions & 2 deletions .github/workflows/update_proto.yml
Original file line number Diff line number Diff line change
Expand Up @@ -19,8 +19,14 @@ jobs:
run: |
find lnd/lnrpc -name '*.proto' -exec bash -c 'test -e proto/`basename {}` && cp {} proto' \;
sed -i 's/^import.*\//import "/' proto/*
git add proto
- name: update generated types
run: |
bash grpc_gen_types.sh
git add src/grpc/types
- name: commit changes
run: |
git config user.name 'Github Actions'
git config user.email [email protected]
git add proto
git commit -m "Update proto files" || echo -n
git commit -m 'Update proto files and types' || echo -n
git push
2 changes: 1 addition & 1 deletion dist/src/controllers/invoices.js.map

Large diffs are not rendered by default.

18 changes: 12 additions & 6 deletions dist/src/controllers/queries.js

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion dist/src/controllers/queries.js.map

Large diffs are not rendered by default.

4 changes: 2 additions & 2 deletions dist/src/crypto/rsa.js

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion dist/src/crypto/rsa.js.map

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

60 changes: 31 additions & 29 deletions dist/src/grpc/greenlight.js

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Loading

0 comments on commit 79d85b6

Please sign in to comment.