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

Multi-chain, multi-denom support for drip #2917

Open
wants to merge 12 commits into
base: main
Choose a base branch
from

fix(drip): cargo lock

3255d50
Select commit
Loading
Failed to load commit list.
Open

Multi-chain, multi-denom support for drip #2917

fix(drip): cargo lock
3255d50
Select commit
Loading
Failed to load commit list.
Garnix CI / check clippy [x86_64-linux] succeeded Oct 18, 2024 in 12m 56s

Run results

Build succeeded

Details

Last 100 lines of logs:

   |                  ^^^^^^^^^^^^^
...
38 |     reconnecting_jsonrpc_ws_client,
   |     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
   |
   = note: `#[warn(unused_imports)]` on by default

warning: unused variable: `path`
   --> voyager/modules/chain/movement/src/main.rs:368:65
    |
368 |     async fn query_ibc_proof(&self, _: &Extensions, at: Height, path: Path) -> RpcResult<Value> {
    |                                                                 ^^^^ help: if this is intentional, prefix it with an underscore: `_path`
    |
    = note: `#[warn(unused_variables)]` on by default

warning: useless conversion to the same type: `std::ops::Range<u64>`
   --> voyager/plugins/event-source/movement/src/main.rs:443:33
    |
443 | / ...                   ((height + 1)..next_height)
444 | | ...                       .into_iter()
    | |______________________________________^ help: consider removing `.into_iter()`: `((height + 1)..next_height)`
    |
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#useless_conversion
    = note: `#[warn(clippy::useless_conversion)]` on by default

warning: `voyager-chain-module-movement` (bin "voyager-chain-module-movement" test) generated 1 warning
warning: `voyager-event-source-plugin-movement` (bin "voyager-event-source-plugin-movement" test) generated 2 warnings (run `cargo clippy --fix --bin "voyager-event-source-plugin-movement" --tests` to apply 2 suggestions)
warning: very complex type used. Consider factoring parts into `type` definitions
   --> hubble/src/postgres.rs:246:13
    |
246 |     tokens: &[(i64, String, String, i64, Option<String>, String)],
    |             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    |
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#type_complexity
    = note: `#[warn(clippy::type_complexity)]` on by default

warning: very complex type used. Consider factoring parts into `type` definitions
   --> hubble/src/postgres.rs:250:83
    |
250 |       let (chain_ids, denoms, display_symbols, decimals, logo_uris, display_names): (
    |  ___________________________________________________________________________________^
251 | |         Vec<i64>,
252 | |         Vec<String>,
253 | |         Vec<String>,
...   |
256 | |         Vec<String>,
257 | |     ) = tokens
    | |_____^
    |
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#type_complexity

warning: unneeded `return` statement
  --> hubble/src/token_list.rs:82:17
   |
82 |                 return Ok(None);
   |                 ^^^^^^^^^^^^^^^
   |
   = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return
   = note: `#[warn(clippy::needless_return)]` on by default
help: remove `return`
   |
82 -                 return Ok(None);
82 +                 Ok(None)
   |

warning: unnecessary `if let` since only the `Ok` variant of the iterator element is used
  --> hubble/src/token_list.rs:91:5
   |
91 |       for result in results {
   |       ^             ------- help: try: `results.into_iter().flatten()`
   |  _____|
   | |
92 | |         if let Ok(Some(token_list)) = result {
93 | |             tokens.push(token_list);
94 | |         }
95 | |     }
   | |_____^
   |
help: ...and remove the `if let` statement in the for loop
  --> hubble/src/token_list.rs:92:9
   |
92 | /         if let Ok(Some(token_list)) = result {
93 | |             tokens.push(token_list);
94 | |         }
   | |_________^
   = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#manual_flatten
   = note: `#[warn(clippy::manual_flatten)]` on by default

warning: `hubble` (bin "hubble" test) generated 4 warnings (run `cargo clippy --fix --bin "hubble" --tests` to apply 1 suggestion)
    Finished `release` profile [optimized] target(s) in 1m 06s
buildPhase completed in 1 minutes 7 seconds
Running phase: installPhase
@nix {"action":"setPhase","phase":"installPhase"}
linking previous artifacts /nix/store/8vxwr0dhs6b1sm2canxi3m1yl152vpsm-workspace-cargo-clippy-deps-0.0.0/target.tar.zst to /nix/store/m8g11an3ic6pmvr7x63np76zxmfs90lx-workspace-cargo-clippy-clippy-0.0.0/target.tar.zst
/*stdin*\            : 32.69%   (   106 MiB =>   34.6 MiB, /nix/store/m8g11an3ic6pmvr7x63np76zxmfs90lx-workspace-cargo-clippy-clippy-0.0.0/target.tar.zst) 
Running phase: fixupPhase
@nix {"action":"setPhase","phase":"fixupPhase"}
shrinking RPATHs of ELF executables and libraries in /nix/store/m8g11an3ic6pmvr7x63np76zxmfs90lx-workspace-cargo-clippy-clippy-0.0.0
checking for references to /build/ in /nix/store/m8g11an3ic6pmvr7x63np76zxmfs90lx-workspace-cargo-clippy-clippy-0.0.0...
patching script interpreter paths in /nix/store/m8g11an3ic6pmvr7x63np76zxmfs90lx-workspace-cargo-clippy-clippy-0.0.0