AElfProject / aelf-web3.js

aelf JavaScript SDK
MIT License
16 stars 24 forks source link

fix: 🐛 fix nested protobuf serialize and deserialize error #72

Closed atom-yang closed 4 years ago

vizipi[bot] commented 4 years ago

Pull request analysis by VIZIPI

Below you will find who is the most qualified team member to review your code. This analysis includes his/her work on the code included in this Pull request, in addition to their experience in code affected by these changes ( partly found within the list of potential missing files below )   Feedback always welcome

Reviewers with knowledge related to these changes

Match % Person Commit Count Common Files
100.00 % 轩辕焮痕 6 3
33.33 % swk 2 1

Potential missing files from this Pull request

files commonly committed with a subset of this pr, but not committed this time. (click to collapse)
FilePercentilerate
package.json66.67 %4 out of 6 times
test/e2e/index.test.js66.67 %4 out of 6 times
src/util/keyStore.js66.67 %4 out of 6 times
jest.node.config.js66.67 %4 out of 6 times
dist/aelf.umd.js66.67 %4 out of 6 times
dist/aelf.cjs.js.map66.67 %4 out of 6 times
examples/node/index.js66.67 %4 out of 6 times
dist/aelf.cjs.js66.67 %4 out of 6 times
build/webpack.node.js66.67 %4 out of 6 times
dist/aelf.umd.js.map66.67 %4 out of 6 times

Committed file ranks

  • 92.95%[yarn.lock]
  • 91.95%[src/contract/contractMethod.js]
  • 68.79%[src/util/formatters.js]