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

refactor(StateVariables): replace SlowUpdates with SharedMutable #4760

Closed
Tracked by #2783
benesjan opened this issue Feb 26, 2024 · 0 comments · Fixed by #5885
Closed
Tracked by #2783

refactor(StateVariables): replace SlowUpdates with SharedMutable #4760

benesjan opened this issue Feb 26, 2024 · 0 comments · Fixed by #5885
Assignees

Comments

@benesjan
Copy link
Contributor

benesjan commented Feb 26, 2024

See this issue for reference on naming.

Furthermore, as we have created a new shared mutable implementation #4761 which is in a good enough state now to be used, we should replace the old slow updates tree entirely. Nuke it from orbit.

Look for "TODO(#4760)" in the codebase.

@LHerskind LHerskind changed the title Rename SlowUpdates to SharedMutable refactor(AztecNr): Rename SlowUpdates to SharedMutable Mar 8, 2024
@LHerskind LHerskind changed the title refactor(AztecNr): Rename SlowUpdates to SharedMutable refactor(StateVariables): Rename SlowUpdates to SharedMutable Mar 8, 2024
@LHerskind LHerskind changed the title refactor(StateVariables): Rename SlowUpdates to SharedMutable refactor(StateVariables): replace SlowUpdates with SharedMutable Apr 15, 2024
nventuro added a commit that referenced this issue Apr 22, 2024
Part of #4760.

This removes all mentions of the slow updates tree from the docs. I
found it simpler to have separate PRs removing docs, updating
dependencies and deleting code since some of those interact with one
another.

There's some to-do's that will be covered by #5508, which will also
feature full-fledged explanations on SharedMutable.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

2 participants