Skip to content

Merge in quartz-decouple and transfers app #54

Merge in quartz-decouple and transfers app

Merge in quartz-decouple and transfers app #54

Triggered via pull request June 26, 2024 17:05
@dangushdangush
synchronize #64
transfers
Status Success
Total duration 2m 22s
Billable time 5m
Artifacts

cosmwasm-basic.yml

on: pull_request
Fit to window
Zoom out
Zoom in

Annotations

26 warnings
used `unwrap()` on a `Result` value: utils/mtcs-intent/src/main.rs#L320
warning: used `unwrap()` on a `Result` value --> utils/mtcs-intent/src/main.rs:320:28 | 320 | println!("{}", AccountId::new("wasm", tm_pk.as_bytes()).unwrap()); | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ | = note: if this value is an `Err`, it will panic = help: consider using `expect()` to provide a better panic message = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#unwrap_used
used `unwrap()` on a `Result` value: utils/mtcs-intent/src/main.rs#L310
warning: used `unwrap()` on a `Result` value --> utils/mtcs-intent/src/main.rs:310:22 | 310 | let sk = XPrv::from_str(&sk_str).unwrap(); | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ | = note: if this value is an `Err`, it will panic = help: consider using `expect()` to provide a better panic message = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#unwrap_used
used `unwrap()` on a `Result` value: utils/mtcs-intent/src/main.rs#L307
warning: used `unwrap()` on a `Result` value --> utils/mtcs-intent/src/main.rs:307:28 | 307 | println!("{}", AccountId::new("wasm", tm_pk.as_bytes()).unwrap()); | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ | = note: if this value is an `Err`, it will panic = help: consider using `expect()` to provide a better panic message = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#unwrap_used
used `unwrap()` on a `Result` value: utils/mtcs-intent/src/main.rs#L298
warning: used `unwrap()` on a `Result` value --> utils/mtcs-intent/src/main.rs:298:26 | 298 | let setoff = decrypt(&sk.to_bytes(), &ciphertext).unwrap(); | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ | = note: if this value is an `Err`, it will panic = help: consider using `expect()` to provide a better panic message = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#unwrap_used
used `unwrap()` on a `Result` value: utils/mtcs-intent/src/main.rs#L296
warning: used `unwrap()` on a `Result` value --> utils/mtcs-intent/src/main.rs:296:30 | 296 | let ciphertext = hex::decode(setoff).unwrap(); | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^ | = note: if this value is an `Err`, it will panic = help: consider using `expect()` to provide a better panic message = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#unwrap_used
used `unwrap()` on a `Result` value: utils/mtcs-intent/src/main.rs#L276
warning: used `unwrap()` on a `Result` value --> utils/mtcs-intent/src/main.rs:276:17 | 276 | d.try_into().unwrap() | ^^^^^^^^^^^^^^^^^^^^^ | = note: if this value is an `Err`, it will panic = help: consider using `expect()` to provide a better panic message = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#unwrap_used
used `unwrap()` on a `Result` value: utils/mtcs-intent/src/main.rs#L255
warning: used `unwrap()` on a `Result` value --> utils/mtcs-intent/src/main.rs:255:25 | 255 | let o = decrypt(&sk.to_bytes(), &ciphertext).unwrap(); | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ | = note: if this value is an `Err`, it will panic = help: consider using `expect()` to provide a better panic message = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#unwrap_used
used `unwrap()` on a `Result` value: utils/mtcs-intent/src/main.rs#L252
warning: used `unwrap()` on a `Result` value --> utils/mtcs-intent/src/main.rs:252:30 | 252 | let ciphertext = hex::decode(obligation).unwrap(); | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ | = note: if this value is an `Err`, it will panic = help: consider using `expect()` to provide a better panic message = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#unwrap_used note: the lint level is defined here --> utils/mtcs-intent/src/main.rs:6:5 | 6 | clippy::unwrap_used, | ^^^^^^^^^^^^^^^^^^^
the borrowed expression implements the required traits: utils/cycles-sync/src/main.rs#L380
warning: the borrowed expression implements the required traits --> utils/cycles-sync/src/main.rs:380:41 | 380 | let mnemonic = Mnemonic::random(&mut OsRng, Default::default()); | ^^^^^^^^^^ help: change this to: `OsRng` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrows_for_generic_args = note: `#[warn(clippy::needless_borrows_for_generic_args)]` on by default
you are explicitly cloning with `.map()`: utils/cycles-sync/src/main.rs#L115
warning: you are explicitly cloning with `.map()` --> utils/cycles-sync/src/main.rs:115:48 | 115 | let (debtor_id, creditor_id) = obligation_user_map | ________________________________________________^ 116 | | .get(obligation_digest) 117 | | .map(Clone::clone) | |______________________________________^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#map_clone = note: `#[warn(clippy::map_clone)]` on by default help: consider calling the dedicated `copied` method | 115 ~ let (debtor_id, creditor_id) = obligation_user_map 116 + .get(obligation_digest).copied() |
you are explicitly cloning with `.map()`: utils/cycles-sync/src/main.rs#L115
warning: you are explicitly cloning with `.map()` --> utils/cycles-sync/src/main.rs:115:48 | 115 | let (debtor_id, creditor_id) = obligation_user_map | ________________________________________________^ 116 | | .get(obligation_digest) 117 | | .map(Clone::clone) | |______________________________________^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#map_clone = note: `#[warn(clippy::map_clone)]` on by default help: consider calling the dedicated `copied` method | 115 ~ let (debtor_id, creditor_id) = obligation_user_map 116 + .get(obligation_digest).copied() |
missing documentation for a struct: core/quartz/src/server.rs#L180
warning: missing documentation for a struct --> core/quartz/src/server.rs:180:1 | 180 | pub struct ProofOfPublication { | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
missing documentation for an associated function: core/quartz/src/server.rs#L52
warning: missing documentation for an associated function --> core/quartz/src/server.rs:52:5 | 52 | pub fn new(config: Config, sk: Arc<Mutex<Option<SigningKey>>>, attestor: A) -> Self { | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
missing documentation for a struct: core/quartz/src/server.rs#L41
warning: missing documentation for a struct --> core/quartz/src/server.rs:41:1 | 41 | pub struct CoreService<A> { | ^^^^^^^^^^^^^^^^^^^^^^^^^
missing documentation for a struct: core/quartz/src/attestor.rs#L42
warning: missing documentation for a struct --> core/quartz/src/attestor.rs:42:1 | 42 | pub struct MockAttestor; | ^^^^^^^^^^^^^^^^^^^^^^^
missing documentation for a struct: core/quartz/src/attestor.rs#L20
warning: missing documentation for a struct --> core/quartz/src/attestor.rs:20:1 | 20 | pub struct EpidAttestor; | ^^^^^^^^^^^^^^^^^^^^^^^
missing documentation for a method: core/quartz/src/attestor.rs#L16
warning: missing documentation for a method --> core/quartz/src/attestor.rs:16:5 | 16 | fn mr_enclave(&self) -> Result<MrEnclave, Self::Error>; | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
missing documentation for a method: core/quartz/src/attestor.rs#L14
warning: missing documentation for a method --> core/quartz/src/attestor.rs:14:5 | 14 | fn quote(&self, user_data: impl HasUserData) -> Result<Vec<u8>, Self::Error>; | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
missing documentation for an associated type: core/quartz/src/attestor.rs#L12
warning: missing documentation for an associated type --> core/quartz/src/attestor.rs:12:5 | 12 | type Error: ToString; | ^^^^^^^^^^^^^^^^^^^^
missing documentation for a trait: core/quartz/src/attestor.rs#L11
warning: missing documentation for a trait --> core/quartz/src/attestor.rs:11:1 | 11 | pub trait Attestor { | ^^^^^^^^^^^^^^^^^^
missing documentation for a module: core/quartz/src/lib.rs#L17
warning: missing documentation for a module --> core/quartz/src/lib.rs:17:1 | 17 | pub mod server; | ^^^^^^^^^^^^^^
missing documentation for a module: core/quartz/src/lib.rs#L16
warning: missing documentation for a module --> core/quartz/src/lib.rs:16:1 | 16 | pub mod attestor; | ^^^^^^^^^^^^^^^^ | note: the lint level is defined here --> core/quartz/src/lib.rs:7:5 | 7 | missing_docs, | ^^^^^^^^^^^^
Schema
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/checkout@v2. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
Schema
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
Test Suite
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/checkout@v2. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
Test Suite
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/