Sanity Check codebase

fixup! plugin: allow to exectute in an async way #588

Triggered via push April 7, 2024 14:46

vincenzopalazzo

pushed 9a2ade3

macros/async-execution

Status Success

Total duration 59s

Artifacts –

build.yml

on: push

Lints

27s

Matrix: Build

Annotations

120 warnings

returning the result of a `let` binding from a block: tests/src/lib.rs#L47

warning: returning the result of a `let` binding from a block --> tests/src/lib.rs:47:9 | 46 | let cln = async_run!(cln::Node::with_params("--developer", "regtest")).unwrap(); | -------------------------------------------------------------------------------- unnecessary `let` binding 47 | cln | ^^^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#let_and_return help: return the expression directly | 46 ~ 47 ~ async_run!(cln::Node::with_params("--developer", "regtest")).unwrap() |

returning the result of a `let` binding from a block: tests/src/lib.rs#L40

warning: returning the result of a `let` binding from a block --> tests/src/lib.rs:40:9 | 39 | ... let cln = async_run!(cln::Node::with_params(&format!("--developer --plugin={pwd}/target/debug/examples/foo_plugin --plugin={pwd}/target/debug/examples/macros_ex"), "regtest")).unwrap(); | ----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- unnecessary `let` binding 40 | ... cln | ^^^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#let_and_return = note: `#[warn(clippy::let_and_return)]` on by default help: return the expression directly | 39 ~ 40 ~ async_run!(cln::Node::with_params(&format!("--developer --plugin={pwd}/target/debug/examples/foo_plugin --plugin={pwd}/target/debug/examples/macros_ex"), "regtest")).unwrap() |

usage of an `Arc` that is not `Send` and `Sync`: testing/src/cln.rs#L85

warning: usage of an `Arc` that is not `Send` and `Sync` --> testing/src/cln.rs:85:19 | 85 | let btc = Arc::new(btc); | ^^^^^^^^^^^^^ | = note: `Arc<BtcNode>` is not `Send` and `Sync` as: = note: - the trait `Sync` is not implemented for `BtcNode` = help: consider using an `Rc` instead. `Arc` does not provide benefits for non `Send` and `Sync` types = note: if you intend to use `Arc` with `Send` and `Sync` traits = note: wrap the inner type with a `Mutex` or implement `Send` and `Sync` for `BtcNode` = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#arc_with_non_send_sync = note: `#[warn(clippy::arc_with_non_send_sync)]` on by default

this `RefCell` reference is held across an `await` point: testing/src/btc.rs#L120

warning: this `RefCell` reference is held across an `await` point --> testing/src/btc.rs:120:24 | 120 | for process in self.process.borrow_mut().iter_mut() { | ^^^^^^^^^^^^^^^^^^^^^^^^^ | = help: ensure the reference is dropped before calling `await` note: these are all the `await` points this reference is held through --> testing/src/btc.rs:121:28 | 121 | process.kill().await?; | ^^^^^ 122 | let _ = process.wait().await?; | ^^^^^ = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#await_holding_refcell_ref = note: `#[warn(clippy::await_holding_refcell_ref)]` on by default

`crate` references the macro call's crate: testing/src/lib.rs#L32

warning: `crate` references the macro call's crate --> testing/src/lib.rs:32:17 | 32 | use crate::DEFAULT_TIMEOUT; | ^^^^^ help: to reference the macro definition's crate, use: `$crate` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#crate_in_macro_def = note: `#[warn(clippy::crate_in_macro_def)]` on by default

this `let...else` may be rewritten with the `?` operator: plugin/src/types.rs#L28

warning: this `let...else` may be rewritten with the `?` operator --> plugin/src/types.rs:28:9 | 28 | / let Some(ref value) = self.value else { 29 | | return None; 30 | | }; | |__________^ help: replace it with: `let ref value = self.value?;` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#question_mark = note: `#[warn(clippy::question_mark)]` on by default

redundant pattern matching, consider using `is_ok()`: plugin/src/plugin.rs#L295

warning: redundant pattern matching, consider using `is_ok()` --> plugin/src/plugin.rs:295:19 | 295 | while let Ok(_) = reader.read_line(&mut buffer).await { | ----------^^^^^-------------------------------------- help: try: `while (reader.read_line(&mut buffer).await).is_ok()` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_pattern_matching = note: `#[warn(clippy::redundant_pattern_matching)]` on by default

using `Option.and_then(|x| Some(y))`, which is more succinctly expressed as `map(|x| y)`: plugin/src/plugin.rs#L155

using `Option.and_then(|x| Some(y))`, which is more succinctly expressed as `map(|x| y)`: plugin/src/plugin.rs#L154

using `Option.and_then(|x| Some(y))`, which is more succinctly expressed as `map(|x| y)`: plugin/src/plugin.rs#L152

non-binding `let` on a future: plugin/src/plugin.rs#L132

warning: non-binding `let` on a future --> plugin/src/plugin.rs:132:9 | 132 | / let _ = tokio::task::spawn(async move { 133 | | writer 134 | | .write_all(serde_json::to_string(&request).unwrap().as_bytes()) 135 | | .await 136 | | .unwrap(); 137 | | writer.flush().await.unwrap(); 138 | | }); | |___________^ | = help: consider awaiting the future or dropping explicitly with `std::mem::drop` = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#let_underscore_future

non-binding `let` on a future: plugin/src/plugin.rs#L84

warning: non-binding `let` on a future --> plugin/src/plugin.rs:84:13 | 84 | / let _ = tokio::task::spawn(async move { 85 | | writer 86 | | .write_all(serde_json::to_string(&request).unwrap().as_bytes()) 87 | | .await 88 | | .unwrap(); 89 | | writer.flush().await.unwrap(); 90 | | }); | |_______________^ | = help: consider awaiting the future or dropping explicitly with `std::mem::drop` = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#let_underscore_future = note: `#[warn(clippy::let_underscore_future)]` on by default

very complex type used. Consider factoring parts into `type` definitions: plugin/src/plugin.rs#L52

warning: very complex type used. Consider factoring parts into `type` definitions --> plugin/src/plugin.rs:52:14 | 52 | on_init: Option<Arc<dyn Fn(&mut Plugin<T>) -> Value>>, | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#type_complexity

the following explicit lifetimes could be elided: 'c: plugin/src/commands/mod.rs#L27

warning: the following explicit lifetimes could be elided: 'c --> plugin/src/commands/mod.rs:27:18 | 27 | fn call_void<'c>(&self, _plugin: &mut Plugin<T>, _request: &'c serde_json::Value) {} | ^^ ^^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_lifetimes = note: `#[warn(clippy::needless_lifetimes)]` on by default help: elide the lifetimes | 27 - fn call_void<'c>(&self, _plugin: &mut Plugin<T>, _request: &'c serde_json::Value) {} 27 + fn call_void(&self, _plugin: &mut Plugin<T>, _request: &serde_json::Value) {} |

this lifetime isn't used in the function definition: plugin/src/commands/mod.rs#L18

warning: this lifetime isn't used in the function definition --> plugin/src/commands/mod.rs:18:13 | 18 | fn call<'c>( | ^^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#extra_unused_lifetimes = note: `#[warn(clippy::extra_unused_lifetimes)]` on by default

very complex type used. Consider factoring parts into `type` definitions: plugin/src/commands/builtin.rs#L54

warning: very complex type used. Consider factoring parts into `type` definitions --> plugin/src/commands/builtin.rs:54:25 | 54 | pub(crate) on_init: Option<Arc<dyn Fn(&mut Plugin<T>) -> Value>>, | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ | = 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

single-character string constant used as pattern: plugin_macros/src/attr_parser.rs#L39

warning: single-character string constant used as pattern --> plugin_macros/src/attr_parser.rs:39:47 | 39 | let value = value.to_string().replace("\"", ""); | ^^^^ help: consider using a `char`: `'"'` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#single_char_pattern = note: `#[warn(clippy::single_char_pattern)]` on by default

this `impl` can be derived: plugin_macros/src/plugin.rs#L61

warning: this `impl` can be derived --> plugin_macros/src/plugin.rs:61:1 | 61 | / impl Default for PluginDeclaration { 62 | | fn default() -> Self { 63 | | Self { 64 | | state: None, ... | 70 | | } 71 | | } | |_^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#derivable_impls = note: `#[warn(clippy::derivable_impls)]` on by default = help: remove the manual implementation... help: ...and instead derive it | 9 + #[derive(Default)] 10 | pub struct PluginDeclaration { |

this expression creates a reference which is immediately dereferenced by the compiler: plugin_macros/src/plugin.rs#L45

warning: this expression creates a reference which is immediately dereferenced by the compiler --> plugin_macros/src/plugin.rs:45:47 | 45 | let mut inner = KTokenStream::new(&inner); | ^^^^^^ help: change this to: `inner` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrow

this expression creates a reference which is immediately dereferenced by the compiler: plugin_macros/src/plugin.rs#L30

warning: this expression creates a reference which is immediately dereferenced by the compiler --> plugin_macros/src/plugin.rs:30:47 | 30 | let mut inner = KTokenStream::new(&inner); | ^^^^^^ help: change this to: `inner` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrow = note: `#[warn(clippy::needless_borrow)]` on by default

Lints

Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/checkout@v2, actions-rs/toolchain@v1, actions-rs/cargo@v1, actions-rs/clippy-check@v1. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.

Lints

The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2, actions-rs/toolchain@v1, actions-rs/cargo@v1, actions-rs/clippy-check@v1. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/

Lints

The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/

Lints

the following packages contain code that will be rejected by a future version of Rust: rstest v0.10.0

Build (stable)

Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/checkout@v2, actions-rs/toolchain@v1, actions-rs/cargo@v1. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.

Build (stable)

The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2, actions-rs/toolchain@v1, actions-rs/cargo@v1. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/

Build (stable)