Skip to content
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

staking: namespace state keys and remove dups #3764

Merged
merged 5 commits into from
Feb 7, 2024
Merged

Conversation

erwanor
Copy link
Member

@erwanor erwanor commented Feb 7, 2024

This is the final chunk of work for #3615.

This PR:

  • remove duplicates in the staking state keys
  • picks a namespacing format that leaves room for future optimization
  • organize state keys into a module hierarchy

Overall, there aren't dramatic changes, but a series of modest attempts at future-proofing our state key format.

@erwanor erwanor changed the title staking: staking: namespace state keys and remove dups Feb 7, 2024
@erwanor erwanor force-pushed the erwan/staking_state_keys branch from aa52172 to 6a47202 Compare February 7, 2024 16:58
@erwanor erwanor force-pushed the erwan/staking_state_keys branch from 7303ba2 to 047fedb Compare February 7, 2024 17:35
@cratelyn cratelyn self-requested a review February 7, 2024 17:57
@conorsch conorsch added this to the Testnet 65: Deimos milestone Feb 7, 2024
@erwanor erwanor merged commit ee1d315 into main Feb 7, 2024
7 checks passed
@erwanor erwanor deleted the erwan/staking_state_keys branch February 7, 2024 20:28
@cratelyn cratelyn added the A-staking Area: Design and implementation of staking and delegation label Feb 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-staking Area: Design and implementation of staking and delegation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants