-
Notifications
You must be signed in to change notification settings - Fork 21
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
Complete feature std-fs-io in casper-client feat-2.0 #153
Conversation
# Conflicts: # Cargo.toml # lib/cli.rs # lib/cli/deploy.rs
…tem/casper-client-rs into rustSDK # Conflicts: # Cargo.toml # lib/cli.rs # lib/cli/parse.rs
@jacek-casper I believe there is no CI/CD running on feat-track-2.0 branch and that we sould probably add it in for this branch for PR because it seems to me the branch might not pass cargo audit and clippy |
You're right @gRoussac , the CI is not enabled here, it might be worth adding the branch to the workflow file: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm to me, but I think it'd be good to get 1 more approval from someone else familiar with the client
@jacek-casper You review was great thanks again, I need the PR on node to be merged first and then I will imply Rafal for this one two before merging. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍
with just a few nits worth double checking
…per-client-rs into rustSDK-feat-2.0 # Conflicts: # lib/rpcs/v1_5_0/speculative_exec.rs
…per-client-rs into rustSDK-feat-2.0
…per-client-rs into rustSDK-feat-2.0 # Conflicts: # .github/workflows/ci-casper-client-rs.yml # Cargo.toml
…per-client-rs into rustSDK-feat-2.0 # Conflicts: # Cargo.toml # lib/cli/tests.rs
Add wasm32-unknown-unknown target
Add default fetaure
Add
std-fs-io
Remove filesystem I/O functionality from the
std
feature, and gate this behind a new featurestd-fs-io
which depends upon std.Allows to compile types without
secret_key_from_file/read_transaction_file/make_transaction/send_transaction_file/speculative_send_transaction_file/sign_transaction_file/json_pretty_print/....
Update crates
Add
with_bytes()
/session_bytes
toSessionStrParams
payment_bytesto
PaymentStrParams`Pub
struct JsonArg
Mutualize code for
get_maybe_secret_key()
Add
lint-no-default-features/check-no-default-features/keygen::run
Add documentation on
args_from_simple_or_json
Add some Clone on exposed result types
Change tokio to
[tokio::main(flavor = "current_thread")]
as dev branch (required)