Summary generated by Reviewpad on 24 Jul 23 15:58 UTC
This pull request renames the FlushAllEntries function to FlushCachesToStore in multiple files. The function is responsible for flushing the cache entries for all stores in the bulkStoreCache struct. The renaming is consistent across the affected files, including ibc_handle_event_test.go, module.go, bulk_store_cache.go, provable_store.go, provable_store_test.go, mock.go, and ibc_store_module.go. The patch also includes some code formatting changes.
Description
Summary generated by Reviewpad on 24 Jul 23 15:58 UTC
This pull request renames the
FlushAllEntries
function toFlushCachesToStore
in multiple files. The function is responsible for flushing the cache entries for all stores in thebulkStoreCache
struct. The renaming is consistent across the affected files, includingibc_handle_event_test.go
,module.go
,bulk_store_cache.go
,provable_store.go
,provable_store_test.go
,mock.go
, andibc_store_module.go
. The patch also includes some code formatting changes.Issue
Fixes https://github.com/pokt-network/pocket/pull/868/files#r1262167821
Type of change
Please mark the relevant option(s):
List of changes
FlushAllEntries
toFlushCachesToStore
FlushEntries
toFlushCache
Testing
make develop_test
; if any code changes were mademake test_e2e
on k8s LocalNet; if any code changes were madee2e-devnet-test
passes tests on DevNet; if any code was changedRequired Checklist
godoc
format comments on touched members (see: tip.golang.org/doc/comment)If Applicable Checklist
shared/docs/*
if I updatedshared/*
README(s)