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

feat: Migrate to no_std #70

Merged
merged 5 commits into from
Jul 9, 2024
Merged

feat: Migrate to no_std #70

merged 5 commits into from
Jul 9, 2024

Conversation

shuhuiluo
Copy link
Collaborator

@shuhuiluo shuhuiluo commented Jul 9, 2024

This commit migrates the codebase to use no_std, shifting various library imports from std to core and alloc. It also removes std::sync::Mutex and a cache intended to store Ether instances. A configuration attribute for disabling std in non-test environments has been added in lib.rs. All necessary changes to enable compatibility with no_std environments have been accounted for.

Closes #69.

This commit migrates the codebase to use `no_std`, shifting various library imports from `std` to `core` and `alloc`. It also removes `std::sync::Mutex` and a cache intended to store Ether instances. A configuration attribute for disabling `std` in non-test environments has been added in `lib.rs`. All necessary changes to enable compatibility with `no_std` environments have been accounted for.
@shuhuiluo shuhuiluo requested a review from malik672 July 9, 2024 08:11
@shuhuiluo shuhuiluo self-assigned this Jul 9, 2024
shuhuiluo added 2 commits July 9, 2024 01:13
The `thiserror` crate has been made an optional dependency in order to support both `std` and `no_std` environments. This change is reflected in the `Cargo.toml` and related Rust files. The SDK version has been updated as a result.
@shuhuiluo shuhuiluo marked this pull request as draft July 9, 2024 08:55
@shuhuiluo shuhuiluo marked this pull request as ready for review July 9, 2024 09:08
@malik672 malik672 merged commit a776ffc into refactor Jul 9, 2024
2 checks passed
@shuhuiluo shuhuiluo deleted the no_std branch July 9, 2024 17:08
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.

2 participants