Skip to content

chore(no_std): fixed no_std compiler errors in reth-primitives-traits#9572

Merged
mattsse merged 6 commits into
paradigmxyz:mainfrom
martinezjorge:fix-no-std-compiler-errors-in-primitive-traits
Jul 17, 2024
Merged

chore(no_std): fixed no_std compiler errors in reth-primitives-traits#9572
mattsse merged 6 commits into
paradigmxyz:mainfrom
martinezjorge:fix-no-std-compiler-errors-in-primitive-traits

Conversation

@martinezjorge
Copy link
Copy Markdown
Contributor

Helps progress #9478

@martinezjorge martinezjorge changed the title chore(no_std): fixed no_std compiler errors in reth-primitive-traits crate chore(no_std): fixed no_std compiler errors in reth-primitives-traits Jul 17, 2024
Copy link
Copy Markdown
Collaborator

@mattsse mattsse left a comment

Choose a reason for hiding this comment

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

what we also need is setting this crate's default-features = false on the workspace level cargo.toml

Comment thread crates/primitives-traits/src/constants/gas_units.rs Outdated
Comment thread crates/primitives-traits/src/lib.rs Outdated
@mattsse mattsse added the C-debt A clean up/refactor of existing code label Jul 17, 2024
@mattsse mattsse requested a review from gakonst as a code owner July 17, 2024 16:44
@mattsse mattsse force-pushed the fix-no-std-compiler-errors-in-primitive-traits branch 2 times, most recently from 7c105bd to fbeaba1 Compare July 17, 2024 16:45
Copy link
Copy Markdown
Collaborator

@mattsse mattsse left a comment

Choose a reason for hiding this comment

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

lgtm,

I did some finishing touchups because this pr is required by something I'm working on.

@mattsse mattsse force-pushed the fix-no-std-compiler-errors-in-primitive-traits branch 2 times, most recently from 4a004f5 to f45bddd Compare July 17, 2024 16:51
@mattsse mattsse enabled auto-merge July 17, 2024 16:53
@mattsse mattsse force-pushed the fix-no-std-compiler-errors-in-primitive-traits branch from f45bddd to e461ec5 Compare July 17, 2024 16:56
@mattsse mattsse requested a review from fgimenez as a code owner July 17, 2024 17:08
@mattsse mattsse added this pull request to the merge queue Jul 17, 2024
Merged via the queue into paradigmxyz:main with commit 8417473 Jul 17, 2024
theochap pushed a commit to ethereum-optimism/optimism that referenced this pull request Jan 22, 2026
…its` (paradigmxyz/reth#9572)

Co-authored-by: Matthias Seitz <matthias.seitz@outlook.de>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

C-debt A clean up/refactor of existing code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants