AElfProject / aelf-web3.js

aelf JavaScript SDK
MIT License
16 stars 24 forks source link

feat: 🎸 pbUtils add deserializeTransactionFee #73

Closed hzz780 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 % atom-yang 15 7

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
test/e2e/index.test.js83.33 %5 out of 6 times
src/util/utils.js83.33 %5 out of 6 times
src/util/formatters.js83.33 %5 out of 6 times
src/util/keyStore.js83.33 %5 out of 6 times
src/contract/index.js83.33 %5 out of 6 times
proto/crosschain.proto83.33 %5 out of 6 times
proto/kernel.proto83.33 %5 out of 6 times
proto/auth.proto.json83.33 %5 out of 6 times
proto/kernel.proto.json83.33 %5 out of 6 times
proto/crosschain.proto.json83.33 %5 out of 6 times

Committed file ranks

  • 5.59%[test/unit/util/proto.test.js]
  • 73.36%[src/util/proto.js]
  • 67.76%[proto/core.proto.json]
  • 67.76%[proto/core.proto]
  • 93.75%[src/contract/contractMethod.js]
  • 5.59%[src/util/specialProto/deserializeTransactionFee.js]
  • 5.59%[proto/transaction_fee.proto.json]