[IBC] chore: Rename FlushAllEntries => FlushCachesToStore #934
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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)