Rename `ckb_chain::consume_unverified` to `ckb_chain::verify` - Githubissues

nervosnetwork / ckb

The Nervos CKB is a public permissionless blockchain, and the layer 1 of Nervos network.

https://www.nervos.org

MIT License

1.15k stars 228 forks source link

Rename `ckb_chain::consume_unverified` to `ckb_chain::verify` #4626

Closed eval-exec closed 1 week ago

eval-exec commented 2 weeks ago

What problem does this PR solve?

In current develop: The log looks like:

2024-08-30 02:31:18.961 +00:00 consume_unverified_blocks INFO ckb_chain::consume_unverified  [verify block] new best block found: 13721606 => 0x26ba9b2e2c562aecdb13042feab7c7c850bacd95748127a43167a104680b3044, difficulty diff = 0x34b5ec519ab53fc6, unverified_tip: 13721606

the ckb_chain::consume_unverified looks inappropriate. Changing it to ckb_chain::verify should be more appropriate.

Rename ckb_chain::consume_unverified to ckb_chain::verify

What is changed and how it works?

What's Changed:

Related changes

Change consume_unverified.rs to verify.rs in ./chain/src/

Check List

Tests

Unit test
Integration test
Manual test (add detailed scripts or steps below)
No code ci-runs-only: [ quick_checks,linters ]

Side effects

None

Release note

None: Exclude this PR from the release note.