Skip to content
This repository has been archived by the owner on Mar 20, 2024. It is now read-only.

darwinia-ethereum custom command #207

Merged
merged 18 commits into from
Jul 27, 2021
Merged

darwinia-ethereum custom command #207

merged 18 commits into from
Jul 27, 2021

Conversation

fewensa
Copy link
Contributor

@fewensa fewensa commented Jul 23, 2021

darwinia-ethereum custom command

  • affirm
  • affirm_force
  • affirm_raw
  • affirmactions
  • confirm
  • ecdsa
  • encrypt_conf
    use encrypt_key
  • encrypt_key
  • guard
  • info_d2e
  • keys
  • show_parcel
  • sign_mmr_root

Related:
#140

replace: #203

@fewensa fewensa mentioned this pull request Jul 23, 2021
13 tasks
@fewensa fewensa mentioned this pull request Jul 27, 2021
13 tasks
@fewensa
Copy link
Contributor Author

fewensa commented Jul 27, 2021

@wuminzhe @xiaoch05 @hackfisher all darwinia-ethereum custom commands have been migrated..
please review and test these commands, the document is here https://github.com/darwinia-network/bridger/blob/master/task/task-darwinia-ethereum/docs/Guide.md#custom-command

@fewensa fewensa marked this pull request as ready for review July 27, 2021 06:11
@hackfisher
Copy link
Contributor

@fewensa Tests failed:

running 4 tests
test src/lib.rs - derive_bridge_crypto (line 14) ... FAILED
test src/lib.rs - derive_bridge_crypto (line 28) ... FAILED
test src/lib.rs - derive_bridge_crypto (line 60) ... FAILED
test src/lib.rs - derive_bridge_crypto (line 71) ... FAILED

failures:

---- src/lib.rs - derive_bridge_crypto (line 14) stdout ----
error: cannot find derive macro `BridgeCrypto` in this scope
 --> src/lib.rs:15:10
  |
3 | #[derive(BridgeCrypto)]
  |          ^^^^^^^^^^^^

error: cannot find attribute `crypto` in this scope
 --> src/lib.rs:17:7
  |
5 |     #[crypto(is_enable)]
  |       ^^^^^^

Xavier Lau and others added 2 commits July 27, 2021 15:20
@hackfisher hackfisher merged commit e06f954 into master Jul 27, 2021
@hackfisher hackfisher deleted the d2e-command branch July 27, 2021 07:49
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants