Skip to content

Removes write version from StorableAccounts#542

Merged
brooksprumo merged 1 commit intoanza-xyz:masterfrom
brooksprumo:write-version/storable-accounts
Apr 3, 2024
Merged

Removes write version from StorableAccounts#542
brooksprumo merged 1 commit intoanza-xyz:masterfrom
brooksprumo:write-version/storable-accounts

Conversation

@brooksprumo
Copy link
Copy Markdown

Problem

Now that we no longer use the write version when storing accounts (see #476), StorableAccounts does not need to care about it.

Summary of Changes

Remove write version from StorableAccounts.

Note, this PR does not change StorableAccountsWithHashesAndWriteVersions. That'll happen in the next PR.

@brooksprumo brooksprumo self-assigned this Apr 2, 2024
@brooksprumo brooksprumo force-pushed the write-version/storable-accounts branch from 4b11dd6 to d0a2de7 Compare April 2, 2024 15:49
@brooksprumo brooksprumo marked this pull request as ready for review April 2, 2024 16:23
Copy link
Copy Markdown

@HaoranYi HaoranYi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

Copy link
Copy Markdown

@jeffwashington jeffwashington left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm
ty

@brooksprumo brooksprumo merged commit ce1f41e into anza-xyz:master Apr 3, 2024
@brooksprumo brooksprumo deleted the write-version/storable-accounts branch April 3, 2024 14:21
OliverNChalk pushed a commit to OliverNChalk/agave that referenced this pull request Nov 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants