-
Notifications
You must be signed in to change notification settings - Fork 1k
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.
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
[Neo Core Store] Rename various snapshots. #3406
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
all are changed?
what you mean? |
if we don't call to the obsolete methods |
Sorry I still dont get that,,, what you mean if we dont call to the absolete methods? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we need to leave the obsolete methods in the code?
Yes, cause people could be using them since they are public, we can not remove them until the verion after the next version. |
* master: Fixed Publish Step (neo-project#3411) Fix release compilation (neo-project#3417) [Neo Plugin UT] Rpcserver unit test on node (neo-project#3353) Improve code coverage (neo-project#3354) `[Add]` Debug Output to `Expect` (neo-project#3407) [Neo Plugin Store] Unit test (neo-project#3399) Bump System.Text.Json from 8.0.3 to 8.0.4 in /src/Neo.Json (neo-project#3416) # Conflicts: # tests/Neo.Plugins.RpcServer.Tests/TestBlockchain.cs # tests/Neo.UnitTests/Network/P2P/Payloads/UT_Header.cs
* master: (27 commits) Revert "Plugin unhandled exception (neo-project#3349)" (neo-project#3366) `[Move]` Part-2 Classes into Different Library - `Neo.IO` (neo-project#3388) [Neo Core Store] Rename various snapshots. (neo-project#3406) [Neo Core Storage] Implicit methods and tests (neo-project#3403) [Neo Core MemoryStore] MemoryStore Unit Tests. (neo-project#3404) Fixed Publish Step (neo-project#3411) Fix release compilation (neo-project#3417) [Neo Plugin UT] Rpcserver unit test on node (neo-project#3353) Improve code coverage (neo-project#3354) `[Add]` Debug Output to `Expect` (neo-project#3407) [Neo Plugin Store] Unit test (neo-project#3399) Bump System.Text.Json from 8.0.3 to 8.0.4 in /src/Neo.Json (neo-project#3416) `[Typo]` Unit Tests - UT_ProtocolSettings.CreateHKSettings (neo-project#3383) `[Fix]` Test Problems (neo-project#3398) Part-1 `Neo.IO` - move (neo-project#3387) Fixed Props Pathing for `dotnet pack` with `nuget` (neo-project#3379) `[neo-cli]` Error Message and Warning - LevelDb (neo-project#3380) Fix crash when comparing ContractPermissionDescriptor (neo-project#3396) `DeprecatedIn` for events (neo-project#3362) Fix download tips (neo-project#3395) ...
Description
The name of snapshot has being misused almost everywhere in the core, snapshotcache is different from snapshot, and clonedcache is also not a snapshot. Misuse the terminology of snapshot confuses developers.
Fixes # (issue)
Type of change
How Has This Been Tested?
Test Configuration:
Checklist: