Process completed with exit code 101.

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

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

warning: usage of `Iterator::fold` on a type that implements `Try` --> partiql-eval/src/eval/expr/path.rs:132:18 | 132 | .fold(Some(owned), |v, path| { | __________________^ 133 | | v.and_then(|v| path.take_val(v, bindings, ctx)) 134 | | }) | |__________________^ help: use `try_fold` instead: `try_fold(owned, |v, path| ...)` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#manual_try_fold

warning: usage of `Iterator::fold` on a type that implements `Try` --> partiql-eval/src/eval/expr/path.rs:125:18 | 125 | .fold(Some(borrowed), |v, path| { | __________________^ 126 | | v.and_then(|v| path.get_val(v, bindings, ctx)) 127 | | }) | |__________________^ help: use `try_fold` instead: `try_fold(borrowed, |v, path| ...)` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#manual_try_fold = note: `#[warn(clippy::manual_try_fold)]` on by default

warning: redundant pattern matching, consider using `is_some()` --> /home/runner/work/partiql-lang-rust/partiql-lang-rust/target/debug/build/partiql-parser-b7f563fc44f4149d/out/partiql.rs:47881:25 | 47881 | let recursive = matches!(rec, Some(_)); | ^^^^^^^^^^^^^^^^^^^^^^ help: try: `rec.is_some()` | = 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

warning: unneeded `return` statement --> extension/partiql-extension-visualize/src/plan_to_dot.rs:209:9 | 209 | return String::from_utf8(output_bytes).expect("invalid utf8"); | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return help: remove `return` | 209 - return String::from_utf8(output_bytes).expect("invalid utf8"); 209 + String::from_utf8(output_bytes).expect("invalid utf8") |

warning: returning the result of a `let` binding from a block --> extension/partiql-extension-visualize/src/plan_to_dot.rs:167:13 | 166 | let expr = expr.replace('>', "\\>"); | ------------------------------------ unnecessary `let` binding 167 | expr | ^^^^ | = 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 | 166 ~ 167 ~ expr.replace('>', "\\>") |

warning: useless use of `format!` --> extension/partiql-extension-visualize/src/plan_to_dot.rs:139:25 | 139 | .set_label(&format!("{label}")); | ^^^^^^^^^^^^^^^^^^ help: consider using `.to_string()`: `label.to_string()` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#useless_format

warning: useless use of `format!` --> extension/partiql-extension-visualize/src/plan_to_dot.rs:112:17 | 112 | format!("{{project * }}") | ^^^^^^^^^^^^^^^^^^^^^^^^^^ help: consider using `.to_string()`: `"{project * }".to_string()` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#useless_format

warning: redundant closure --> extension/partiql-extension-visualize/src/plan_to_dot.rs:97:30 | 97 | .map(|e| expr_to_str(e)) | ^^^^^^^^^^^^^^^^^^ help: replace the closure with the function itself: `expr_to_str` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_closure = note: `#[warn(clippy::redundant_closure)]` on by default

warning: this expression creates a reference which is immediately dereferenced by the compiler --> extension/partiql-extension-visualize/src/plan_to_dot.rs:77:67 | 77 | .map(|e| format!("offset {}", expr_to_str(&e))), | ^^ help: change this to: `e` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrow

warning: this expression creates a reference which is immediately dereferenced by the compiler --> extension/partiql-extension-visualize/src/plan_to_dot.rs:74:66 | 74 | .map(|e| format!("limit {}", expr_to_str(&e))), | ^^ help: change this to: `e` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrow

warning: this `impl` can be derived --> extension/partiql-extension-visualize/src/plan_to_dot.rs:13:1 | 13 | / impl Default for PlanToDot { 14 | | fn default() -> Self { 15 | | PlanToDot {} 16 | | } 17 | | } | |_^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#derivable_impls = help: remove the manual implementation... help: ...and instead derive it | 11 + #[derive(Default)] 12 | pub struct PlanToDot {} |

warning: useless use of `format!` --> extension/partiql-extension-visualize/src/ast_to_dot.rs:458:45 | 458 | CaseSensitivity::CaseInsensitive => format!("{}", sym.value), | ^^^^^^^^^^^^^^^^^^^^^^^^ help: consider using `.to_string()`: `sym.value.to_string()` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#useless_format = note: `#[warn(clippy::useless_format)]` on by default

warning: this expression creates a reference which is immediately dereferenced by the compiler --> extension/partiql-extension-visualize/src/ast_to_dot.rs:136:43 | 136 | Some(ast) => self.to_dot(out, &ast), | ^^^^ help: change this to: `ast` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrow

warning: this expression creates a reference which is immediately dereferenced by the compiler --> extension/partiql-extension-visualize/src/ast_to_dot.rs:123:41 | 123 | res.extend(self.to_dot(out, &ast)); | ^^^^ help: change this to: `ast` | = 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

warning: unneeded `return` statement --> extension/partiql-extension-visualize/src/ast_to_dot.rs:99:9 | 99 | return String::from_utf8(output_bytes).expect("invalid utf8"); | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ | = 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` | 99 - return String::from_utf8(output_bytes).expect("invalid utf8"); 99 + String::from_utf8(output_bytes).expect("invalid utf8") |

warning: this `impl` can be derived --> extension/partiql-extension-visualize/src/ast_to_dot.rs:62:1 | 62 | / impl Default for AstToDot { 63 | | fn default() -> Self { 64 | | AstToDot {} 65 | | } 66 | | } | |_^ | = 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 | 60 + #[derive(Default)] 61 | pub struct AstToDot {} |

warning: the borrowed expression implements the required traits --> extension/partiql-extension-visualize/src/ast_to_dot.rs:52:29 | 52 | out.edge(&from, &target).attributes().set_label(lbl); | ^^^^^^^ help: change this to: `target` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrows_for_generic_args

warning: the borrowed expression implements the required traits --> extension/partiql-extension-visualize/src/ast_to_dot.rs:52:22 | 52 | out.edge(&from, &target).attributes().set_label(lbl); | ^^^^^ help: change this to: `from` | = 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

warning: single-character string constant used as pattern --> extension/partiql-extension-visualize/src/ast_to_dot.rs:32:48 | 32 | cluster.set("label", lbl, lbl.contains(" ")); | ^^^ help: try using a `char` instead: `' '` | = 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

warning: constants have by default a `'static` lifetime --> extension/partiql-extension-visualize/src/common.rs:6:29 | 6 | pub(crate) const FG_COLOR: &'static str = "\"#839496\""; | -^^^^^^^---- help: consider removing `'static`: `&str` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_static_lifetimes = note: `#[warn(clippy::redundant_static_lifetimes)]` on by default

warning: usage of `Iterator::fold` on a type that implements `Try` --> partiql-eval/src/eval/expr/path.rs:132:18 | 132 | .fold(Some(owned), |v, path| { | __________________^ 133 | | v.and_then(|v| path.take_val(v, bindings, ctx)) 134 | | }) | |__________________^ help: use `try_fold` instead: `try_fold(owned, |v, path| ...)` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#manual_try_fold

warning: usage of `Iterator::fold` on a type that implements `Try` --> partiql-eval/src/eval/expr/path.rs:125:18 | 125 | .fold(Some(borrowed), |v, path| { | __________________^ 126 | | v.and_then(|v| path.get_val(v, bindings, ctx)) 127 | | }) | |__________________^ help: use `try_fold` instead: `try_fold(borrowed, |v, path| ...)` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#manual_try_fold = note: `#[warn(clippy::manual_try_fold)]` on by default

warning: redundant pattern matching, consider using `is_some()` --> /home/runner/work/partiql-lang-rust/partiql-lang-rust/target/debug/build/partiql-parser-b7f563fc44f4149d/out/partiql.rs:47881:25 | 47881 | let recursive = matches!(rec, Some(_)); | ^^^^^^^^^^^^^^^^^^^^^^ help: try: `rec.is_some()` | = 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

warning: unneeded `return` statement --> extension/partiql-extension-visualize/src/plan_to_dot.rs:209:9 | 209 | return String::from_utf8(output_bytes).expect("invalid utf8"); | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return help: remove `return` | 209 - return String::from_utf8(output_bytes).expect("invalid utf8"); 209 + String::from_utf8(output_bytes).expect("invalid utf8") |

warning: returning the result of a `let` binding from a block --> extension/partiql-extension-visualize/src/plan_to_dot.rs:167:13 | 166 | let expr = expr.replace('>', "\\>"); | ------------------------------------ unnecessary `let` binding 167 | expr | ^^^^ | = 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 | 166 ~ 167 ~ expr.replace('>', "\\>") |

warning: useless use of `format!` --> extension/partiql-extension-visualize/src/plan_to_dot.rs:139:25 | 139 | .set_label(&format!("{label}")); | ^^^^^^^^^^^^^^^^^^ help: consider using `.to_string()`: `label.to_string()` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#useless_format

warning: useless use of `format!` --> extension/partiql-extension-visualize/src/plan_to_dot.rs:112:17 | 112 | format!("{{project * }}") | ^^^^^^^^^^^^^^^^^^^^^^^^^^ help: consider using `.to_string()`: `"{project * }".to_string()` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#useless_format

warning: redundant closure --> extension/partiql-extension-visualize/src/plan_to_dot.rs:97:30 | 97 | .map(|e| expr_to_str(e)) | ^^^^^^^^^^^^^^^^^^ help: replace the closure with the function itself: `expr_to_str` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_closure = note: `#[warn(clippy::redundant_closure)]` on by default

warning: this expression creates a reference which is immediately dereferenced by the compiler --> extension/partiql-extension-visualize/src/plan_to_dot.rs:77:67 | 77 | .map(|e| format!("offset {}", expr_to_str(&e))), | ^^ help: change this to: `e` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrow

warning: this expression creates a reference which is immediately dereferenced by the compiler --> extension/partiql-extension-visualize/src/plan_to_dot.rs:74:66 | 74 | .map(|e| format!("limit {}", expr_to_str(&e))), | ^^ help: change this to: `e` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrow

warning: this `impl` can be derived --> extension/partiql-extension-visualize/src/plan_to_dot.rs:13:1 | 13 | / impl Default for PlanToDot { 14 | | fn default() -> Self { 15 | | PlanToDot {} 16 | | } 17 | | } | |_^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#derivable_impls = help: remove the manual implementation... help: ...and instead derive it | 11 + #[derive(Default)] 12 | pub struct PlanToDot {} |

warning: useless use of `format!` --> extension/partiql-extension-visualize/src/ast_to_dot.rs:458:45 | 458 | CaseSensitivity::CaseInsensitive => format!("{}", sym.value), | ^^^^^^^^^^^^^^^^^^^^^^^^ help: consider using `.to_string()`: `sym.value.to_string()` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#useless_format = note: `#[warn(clippy::useless_format)]` on by default

warning: this expression creates a reference which is immediately dereferenced by the compiler --> extension/partiql-extension-visualize/src/ast_to_dot.rs:136:43 | 136 | Some(ast) => self.to_dot(out, &ast), | ^^^^ help: change this to: `ast` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrow

warning: this expression creates a reference which is immediately dereferenced by the compiler --> extension/partiql-extension-visualize/src/ast_to_dot.rs:123:41 | 123 | res.extend(self.to_dot(out, &ast)); | ^^^^ help: change this to: `ast` | = 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

warning: unneeded `return` statement --> extension/partiql-extension-visualize/src/ast_to_dot.rs:99:9 | 99 | return String::from_utf8(output_bytes).expect("invalid utf8"); | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ | = 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` | 99 - return String::from_utf8(output_bytes).expect("invalid utf8"); 99 + String::from_utf8(output_bytes).expect("invalid utf8") |

warning: this `impl` can be derived --> extension/partiql-extension-visualize/src/ast_to_dot.rs:62:1 | 62 | / impl Default for AstToDot { 63 | | fn default() -> Self { 64 | | AstToDot {} 65 | | } 66 | | } | |_^ | = 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 | 60 + #[derive(Default)] 61 | pub struct AstToDot {} |

warning: the borrowed expression implements the required traits --> extension/partiql-extension-visualize/src/ast_to_dot.rs:52:29 | 52 | out.edge(&from, &target).attributes().set_label(lbl); | ^^^^^^^ help: change this to: `target` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrows_for_generic_args

warning: the borrowed expression implements the required traits --> extension/partiql-extension-visualize/src/ast_to_dot.rs:52:22 | 52 | out.edge(&from, &target).attributes().set_label(lbl); | ^^^^^ help: change this to: `from` | = 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

warning: single-character string constant used as pattern --> extension/partiql-extension-visualize/src/ast_to_dot.rs:32:48 | 32 | cluster.set("label", lbl, lbl.contains(" ")); | ^^^ help: try using a `char` instead: `' '` | = 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

warning: constants have by default a `'static` lifetime --> extension/partiql-extension-visualize/src/common.rs:6:29 | 6 | pub(crate) const FG_COLOR: &'static str = "\"#839496\""; | -^^^^^^^---- help: consider removing `'static`: `&str` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_static_lifetimes = note: `#[warn(clippy::redundant_static_lifetimes)]` on by default

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

The following actions uses node12 which is deprecated and will be forced to run on node16: 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/

Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/checkout@v3, actions/cache@v3, actions/upload-artifact@v3. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.

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