Skip to content

Conversation

@yiweichi
Copy link
Member

@yiweichi yiweichi commented Jan 11, 2026

This PR enables reth to use legacy state root tree by default.

Fixes #468

@codspeed-hq
Copy link

codspeed-hq bot commented Jan 11, 2026

CodSpeed Performance Report

Merging this PR will not alter performance

Comparing feat-enable-legacy-state-root-by-default (fe5289c) with main (c00bd01)

Summary

✅ 2 untouched benchmarks

Copy link
Collaborator

@frisitano frisitano left a comment

Choose a reason for hiding this comment

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

Let's remove rust-toolchain.toml and update the Makefile:

NIGHTLY_TOOLCHAIN := nightly-2026-01-05

.PHONY: fmt
fmt:
	cargo +$(NIGHTLY_TOOLCHAIN) fmt

.PHONY: lint-toml
lint-toml: ensure-dprint
	dprint fmt

ensure-dprint:
	@if ! command -v dprint &> /dev/null; then \
		echo "dprint not found. Please install it by running the command `cargo install --locked dprint` or refer to the following link for more information: https://github.com/dprint/dprint" \
		exit 1; \
    fi

.PHONY: clippy
clippy:
	cargo +$(NIGHTLY_TOOLCHAIN) clippy \
	--workspace \
	--lib \
	--examples \
	--tests \
	--benches \
	--all-features \
	-- -D warnings

.PHONY: clippy-fix
clippy-fix:
	cargo +$(NIGHTLY_TOOLCHAIN) clippy \
	--workspace \
	--lib \
	--examples \
	--tests \
	--benches \
	--all-features \
	--fix \
	-- -D warnings

.PHONY: udeps
udeps:
	cargo +$(NIGHTLY_TOOLCHAIN) udeps --workspace --lib --examples --tests --benches --all-features --locked

@frisitano frisitano merged commit 567cc38 into main Jan 12, 2026
15 checks passed
@frisitano frisitano deleted the feat-enable-legacy-state-root-by-default branch January 12, 2026 13:24
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.

Enable legacy state root by default

3 participants