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.

Sign up for GitHub

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

Jump to bottom

Fix `fn get_fee_rate_statistics` return type #93

Merged

quake merged 1 commit into nervosnetwork:master from eval-exec:patch-1

Oct 25, 2023

Contributor

eval-exec commented Oct 19, 2023 •

edited

Loading

Ref: https://github.com/nervosnetwork/ckb/blob/cff0cabc02b7d841053f818330a5e9d07c863130/rpc/src/module/chain.rs#L1605-L1607

The first argument of get_fee_rate_statistics should be target, and its return type should be an Option

@quake Request for review, and we need to publish a new cargo release.


          Fix typo, Fix get_fee_rate_statistics return type

5b24731

gpBlockchain mentioned this pull request

mock rpc test result #94

Closed

quake approved these changes

View reviewed changes

quake merged commit f296e77 into nervosnetwork:master

7 checks passed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet