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.

Sign up for GitHub

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

Jump to bottom

docs: remove SMT #1552

Merged
merged 2 commits into from
Mar 29, 2023
Merged

docs: remove SMT #1552

merged 2 commits into from
Mar 29, 2023

Conversation

rootulp
Copy link
Collaborator

@rootulp rootulp commented Mar 28, 2023

Closes #1488

@rootulp rootulp added the specs directly relevant to the specs label Mar 28, 2023
@rootulp rootulp self-assigned this Mar 28, 2023
@MSevey MSevey requested a review from a team March 28, 2023 10:03
@github-actions
Copy link

github-actions bot commented Mar 28, 2023 •
edited

PR Preview Action v1.3.0
🚀 Deployed preview to https://celestiaorg.github.io/celestia-app/pr-preview/pr-1552/
on branch gh-pages at 2023-03-28 12:55 UTC

adlerjohn
adlerjohn reviewed Mar 28, 2023
View reviewed changes
Copy link
Member

@adlerjohn adlerjohn left a comment

Choose a reason for hiding this comment

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

There are references to the SMT under the State section. Should those be removed?

@rootulp rootulp requested a review from adlerjohn March 28, 2023 12:55
@MSevey MSevey requested a review from a team March 28, 2023 12:55
@rootulp rootulp merged commit 10a526d into main Mar 29, 2023
@rootulp rootulp deleted the rp/remove-smt branch March 29, 2023 08:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@evan-forbes evan-forbes evan-forbes approved these changes

@liamsi liamsi Awaiting requested review from liamsi liamsi is a code owner

@MSevey MSevey Awaiting requested review from MSevey

@adlerjohn adlerjohn Awaiting requested review from adlerjohn

Assignees

@rootulp rootulp

Labels
specs directly relevant to the specs
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

specs: consider removing Sparse Merkle Tree section
3 participants
@rootulp @adlerjohn @evan-forbes